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: TestServerShutdown_h2 failures #56036

Closed
gopherbot opened this issue Oct 4, 2022 · 3 comments
Closed

net/http: TestServerShutdown_h2 failures #56036

gopherbot opened this issue Oct 4, 2022 · 3 comments
Labels
FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.

Comments

@gopherbot
Copy link

#!watchflakes
post <- pkg == "net/http" && test == "TestServerShutdown_h2"

Issue created automatically to collect these failures.

Example (log):

--- FAIL: TestServerShutdown_h2 (0.05s)
    serve_test.go:5677: second request should fail. server should be shut down
panic: test timed out after 3m0s

goroutine 10885 [running]:
panic({0x84a320, 0xc0011aa020})
	/workdir/go/src/runtime/panic.go:987 +0x3bb fp=0xc000b5cf98 sp=0xc000b5ced8 pc=0x43751b
testing.(*M).startAlarm.func1()
	/workdir/go/src/testing/testing.go:2107 +0x8e fp=0xc000b5cfe0 sp=0xc000b5cf98 pc=0x5001ee
runtime.goexit()
	/workdir/go/src/runtime/asm_amd64.s:1594 +0x1 fp=0xc000b5cfe8 sp=0xc000b5cfe0 pc=0x46c561
created by time.goFunc
	/workdir/go/src/time/sleep.go:176 +0x32

watchflakes

@gopherbot gopherbot added the NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. label Oct 4, 2022
@gopherbot
Copy link
Author

Found new dashboard test flakes for:

#!watchflakes
post <- pkg == "net/http" && test == "TestServerShutdown_h2"
2022-09-30 03:08 linux-amd64-alpine go@9faf6b79 net/http.TestServerShutdown_h2 (log)
--- FAIL: TestServerShutdown_h2 (0.05s)
    serve_test.go:5677: second request should fail. server should be shut down
panic: test timed out after 3m0s

goroutine 10885 [running]:
panic({0x84a320, 0xc0011aa020})
	/workdir/go/src/runtime/panic.go:987 +0x3bb fp=0xc000b5cf98 sp=0xc000b5ced8 pc=0x43751b
testing.(*M).startAlarm.func1()
	/workdir/go/src/testing/testing.go:2107 +0x8e fp=0xc000b5cfe0 sp=0xc000b5cf98 pc=0x5001ee
runtime.goexit()
	/workdir/go/src/runtime/asm_amd64.s:1594 +0x1 fp=0xc000b5cfe8 sp=0xc000b5cfe0 pc=0x46c561
created by time.goFunc
	/workdir/go/src/time/sleep.go:176 +0x32

watchflakes

@bcmills
Copy link
Contributor

bcmills commented Oct 4, 2022

Compare #38305, #54758 (comment).

@neild
Copy link
Contributor

neild commented Nov 17, 2022

Duplicate of #56421

@neild neild marked this as a duplicate of #56421 Nov 17, 2022
@neild neild closed this as completed Nov 17, 2022
@golang golang locked and limited conversation to collaborators Nov 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Projects
Status: Done
Development

No branches or pull requests

3 participants