Skip to content
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/http: flaky TestTransportReuseConnection_Gzip_ContentLength #22212

Closed
rsc opened this issue Oct 11, 2017 · 2 comments
Closed

net/http: flaky TestTransportReuseConnection_Gzip_ContentLength #22212

rsc opened this issue Oct 11, 2017 · 2 comments
Labels
FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done. Testing An issue that has been verified to require only test changes, not just a test failure.
Milestone

Comments

@rsc
Copy link
Contributor

rsc commented Oct 11, 2017

trybot failure in a cmd/go CL. nothing i'm changing in cmd/go could cause this.
only happened in the race builder, so maybe scheduling-sensitive.

https://storage.googleapis.com/go-build-log/00c78809/linux-amd64-race_9ed057c7.log

--- FAIL: TestTransportReuseConnection_Gzip_ContentLength (0.19s)
	transport_test.go:3411: didn't reuse connection
FAIL
FAIL	net/http	7.526s
2017/10/11 05:02:11 Failed: exit status 1
@rsc rsc added this to the Go1.10 milestone Oct 11, 2017
@bradfitz bradfitz self-assigned this Nov 27, 2017
@bradfitz bradfitz added NeedsFix The path to resolution is known, but the work has not been done. Testing An issue that has been verified to require only test changes, not just a test failure. labels Nov 27, 2017
@bradfitz
Copy link
Contributor

Unfortunately I've been unable to reproduce this (on linux-amd64 or linux-amd64-race), even with high -count values and various -cpu.

/cc @tombergan

@bradfitz
Copy link
Contributor

I just spent some time on this again and still can't reproduce. I also looked at the git history for clues, but nothing.

Closing for now until we see it again or have more clues.

@golang golang locked and limited conversation to collaborators May 29, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done. Testing An issue that has been verified to require only test changes, not just a test failure.
Projects
None yet
Development

No branches or pull requests

3 participants