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

x/net/http2: apparent deadlock in TestTransportResponseHeaderTimeout_Body on windows-386-2008 #52655

Open
bcmills opened this issue May 2, 2022 · 3 comments
Labels
NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Milestone

Comments

@bcmills
Copy link
Contributor

bcmills commented May 2, 2022

panic: test timed out after 10m0s
…
goroutine 10660 [semacquire, 9 minutes]:
sync.runtime_Semacquire(0x13397808)
	C:/workdir/go/src/runtime/sema.go:56 +0x36
sync.(*WaitGroup).Wait(0x13397800)
	C:/workdir/go/src/sync/waitgroup.go:136 +0x87
golang.org/x/net/http2.(*clientTester).run(0x130e9110)
	C:/workdir/gopath/src/golang.org/x/net/http2/transport_test.go:860 +0x190
golang.org/x/net/http2.testTransportResponseHeaderTimeout(0x130ea0f0, 0x1)
	C:/workdir/gopath/src/golang.org/x/net/http2/transport_test.go:2216 +0x151
golang.org/x/net/http2.TestTransportResponseHeaderTimeout_Body(0x130ea0f0)
	C:/workdir/gopath/src/golang.org/x/net/http2/transport_test.go:2164 +0x26
testing.tRunner(0x130ea0f0, 0x823e98)
	C:/workdir/go/src/testing/testing.go:1449 +0x113
created by testing.(*T).Run
	C:/workdir/go/src/testing/testing.go:1496 +0x374

greplogs -l -e 'panic: test timed out after (?:.*\n)*golang\.org/x/net/http2\.TestTransportResponseHeaderTimeout' --since=2021-01-01
2022-05-01T00:24:25-2871e0c-d09ca2c/windows-386-2008

(CC @neild @tombergan)

@bcmills
Copy link
Contributor Author

bcmills commented May 2, 2022

Since we've only seen one of these so far, I'm leaving this on the backlog in order to collect more data — it isn't clear to me whether this failure mode is specific to Windows (in my estimation somewhat likely), specific to 386 (fairly unlikely), or completely platform-dependent (fairly likely).

@bcmills bcmills added the NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. label May 2, 2022
@bcmills bcmills added this to the Backlog milestone May 2, 2022
@Jorropo

This comment was marked as off-topic.

@bcmills
Copy link
Contributor Author

bcmills commented May 3, 2022

@Jorropo, this failure was observed on the x/net builder, so the version of x/net vendored into the main repo should not be relevant. 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Projects
None yet
Development

No branches or pull requests

2 participants