]> Cypherpunks.ru repositories - gostls13.git/commitdiff
misc/cgo/test: use fewer threads in TestSetgidStress in long mode
authorCherry Mui <cherryyz@google.com>
Wed, 27 Jul 2022 16:59:16 +0000 (12:59 -0400)
committerCherry Mui <cherryyz@google.com>
Wed, 27 Jul 2022 20:21:13 +0000 (20:21 +0000)
TestSetgidStress originally spawns 1000 threads for stress testing.
It caused timeout on some builders so CL 415677 reduced to 50 in
short mode. But it still causes flaky timeouts in longtest
builders, so reduce the number of threads in long mode as well.

Should fix #53641.

Change-Id: I02f4ef8a143bb1faafe3d11ad223f36f5cc245c6
Reviewed-on: https://go-review.googlesource.com/c/go/+/419453
Reviewed-by: Bryan Mills <bcmills@google.com>
Run-TryBot: Cherry Mui <cherryyz@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>

misc/cgo/test/setgid2_linux.go

index 9069cff334368a6a2333dd8762fe255b2d661f28..438f5ae512d5fd94d2b73b7782f0cd0393945d8a 100644 (file)
@@ -20,10 +20,7 @@ import (
 )
 
 func testSetgidStress(t *testing.T) {
-       var N = 1000
-       if testing.Short() {
-               N = 50
-       }
+       const N = 50
        ch := make(chan int, N)
        for i := 0; i < N; i++ {
                go func() {