From 462b78fe7027ef0d2e2b40c3cfd1f5a37d307310 Mon Sep 17 00:00:00 2001 From: Cherry Mui Date: Wed, 27 Jul 2022 12:59:16 -0400 Subject: [PATCH] misc/cgo/test: use fewer threads in TestSetgidStress in long mode 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 Run-TryBot: Cherry Mui TryBot-Result: Gopher Robot --- misc/cgo/test/setgid2_linux.go | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/misc/cgo/test/setgid2_linux.go b/misc/cgo/test/setgid2_linux.go index 9069cff334368a..438f5ae512d5fd 100644 --- a/misc/cgo/test/setgid2_linux.go +++ b/misc/cgo/test/setgid2_linux.go @@ -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() {