diff options
author | Cherry Mui <cherryyz@google.com> | 2022-07-01 12:56:39 -0400 |
---|---|---|
committer | Cherry Mui <cherryyz@google.com> | 2022-07-06 14:23:50 +0000 |
commit | 4484c30f788835d8dda0afcefdb12e4b25b2c312 (patch) | |
tree | 47f702c1009613bede9948426f22a12a8ff36764 | |
parent | 2007599dc83aff17d8261338e8d2ab1f2c518a9b (diff) | |
download | go-4484c30f788835d8dda0afcefdb12e4b25b2c312.tar.gz go-4484c30f788835d8dda0afcefdb12e4b25b2c312.zip |
misc/cgo/test: make TestSetgidStress cheaper
TestSetgidStress spawns 1000 threads, which can be expensive on
some platforms or slow builders. Run with 50 threads in short
mode instead.
This makes the failure less reproducible even with buggy code. But
one can manually stress test it (e.g. when a flaky failure appear
on the builder).
Fixes #53641.
Change-Id: I33b5ea5ecaa8c7a56f59c16f9171657ee295db47
Reviewed-on: https://go-review.googlesource.com/c/go/+/415677
Reviewed-by: Austin Clements <austin@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: Cherry Mui <cherryyz@google.com>
-rw-r--r-- | misc/cgo/test/setgid2_linux.go | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/misc/cgo/test/setgid2_linux.go b/misc/cgo/test/setgid2_linux.go index d239893f43..9069cff334 100644 --- a/misc/cgo/test/setgid2_linux.go +++ b/misc/cgo/test/setgid2_linux.go @@ -20,7 +20,10 @@ import ( ) func testSetgidStress(t *testing.T) { - const N = 1000 + var N = 1000 + if testing.Short() { + N = 50 + } ch := make(chan int, N) for i := 0; i < N; i++ { go func() { |