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
net: TestDialParallel is flaky on windows-amd64-longtest #35616
Comments
Each failure is only for some or all of the tests that use In the case above the measured Looks like the use 95 milliseconds was added in patchset 8 of https://golang.org/cl/8768, along with the comment
For the four failures listed above, the largest I can't see any special meaning to |
Change https://golang.org/cl/207466 mentions this issue: |
@gopherbot Please consider for backport to 1.13. This is merely a test fix for a flaky test. It's needed to help #29252. |
Backport issue(s) opened: #39538 (for 1.13). Remember to create the cherry-pick CL(s) as soon as the patch is submitted to master, according to https://golang.org/wiki/MinorReleases. |
Change https://golang.org/cl/237602 mentions this issue: |
…l on Windows For #35616. Fixes #39538. For #29252. Change-Id: I51b2490100cfe0e902da09eee8d027e0ec86ed53 Reviewed-on: https://go-review.googlesource.com/c/go/+/207466 Run-TryBot: Ian Lance Taylor <iant@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Bryan C. Mills <bcmills@google.com> Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org> (cherry picked from commit c20b71e) Reviewed-on: https://go-review.googlesource.com/c/go/+/237602 Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org> Reviewed-by: Alexander Rakoczy <alex@golang.org>
net.TestDialParallel is flaky on the
windows-amd64-longtest
builder.As far as I can tell, the problem is just unexpected timing jitter:
2019-11-15T02:31:58-498eaee/windows-amd64-longtest
2019-11-13T15:52:21-bf49905/windows-amd64-longtest
2019-11-12T01:09:40-c2edcf4/windows-amd64-longtest
2019-11-04T18:51:13-bf7e55b/windows-amd64-longtest
CC @bradfitz @mikioh @ianlancetaylor @zx2c4 @alexbrainman
The text was updated successfully, but these errors were encountered: