-
Notifications
You must be signed in to change notification settings - Fork 17.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
cmd/cgo: TestParallelSleep test is flaky #6636
Labels
Milestone
Comments
What environment are you running in? Is this from running the test in make.bash? Is your system heavily loaded? On my Ubuntu laptop I'm seeing only issue1560.go:44: difference in start time for two sleep(1) is 18ms and the number is never higher than 20 even when I put load on the system. I'm not sure how it's getting up to 562ms for you. |
I have been running all.bash in a continuous loop for several days with GOGC set to 1. This has happened at least 3 times so far. Here are the relevant lines from my log file --- FAIL: TestParallelSleep (1.68 seconds) issue1560.go:44: difference in start time for two sleep(1) is 683ms issue1560.go:48: parallel 1-second sleeps slept for 0.683000 seconds --- FAIL: TestParallelSleep (1.61 seconds) issue1560.go:44: difference in start time for two sleep(1) is 606ms issue1560.go:48: parallel 1-second sleeps slept for 0.606000 seconds --- FAIL: TestParallelSleep (1.56 seconds) issue1560.go:44: difference in start time for two sleep(1) is 562ms issue1560.go:48: parallel 1-second sleeps slept for 0.562000 seconds |
Carl, what do you get when you run this: cd $GOROOT/misc/cgo/test GOGC=1 go test -run TestParallelSleep -v It would be nice to know what the typical execution is, so that we can see how much of a slowdown is happening in the cases that time out. Three failures in 3+ days on a continuous loop is rare enough that I think we can leave it for after Go 1.2. Labels changed: removed go1.2maybe. |
Another one http://build.golang.org/log/bee2f840d3c60e15c1742fdaa30c37990246a4ce Alex |
Last update 2014. Assuming this is no longer an active concern. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
The text was updated successfully, but these errors were encountered: