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

sync: test timeout with -cpu=10 #19476

Closed
mundaym opened this issue Mar 9, 2017 · 0 comments
Closed

sync: test timeout with -cpu=10 #19476

mundaym opened this issue Mar 9, 2017 · 0 comments
Labels
FrozenDueToAge help wanted NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. Testing An issue that has been verified to require only test changes, not just a test failure.
Milestone

Comments

@mundaym
Copy link
Member

mundaym commented Mar 9, 2017

Seen on a trybot run:

https://storage.googleapis.com/go-build-log/8385b08e/linux-386_d0ab7d46.log

##### sync -cpu=10
panic: test timed out after 2m0s

goroutine 3628 [running]:
testing.startAlarm.func1()
	/tmp/workdir/go/src/testing/testing.go:1029 +0xd1
created by time.goFunc
	/tmp/workdir/go/src/time/sleep.go:170 +0x37

goroutine 1 [chan receive]:
testing.(*T).Run(0x18516090, 0x815b76b, 0x14, 0x8161da4, 0x80e4a01)
	/tmp/workdir/go/src/testing/testing.go:702 +0x276
testing.runTests.func1(0x18516090)
	/tmp/workdir/go/src/testing/testing.go:888 +0x4d
testing.tRunner(0x18516090, 0x18549f04)
	/tmp/workdir/go/src/testing/testing.go:659 +0x79
testing.runTests(0x1850a170, 0x81fce00, 0x1a, 0x1a, 0x81fe380)
	/tmp/workdir/go/src/testing/testing.go:886 +0x236
testing.(*M).Run(0x1854df98, 0x0)
	/tmp/workdir/go/src/testing/testing.go:828 +0xc0
main.main()
	sync/_test/_testmain.go:168 +0xc6

goroutine 3647 [semacquire]:
sync.runtime_Semacquire(0x1868a0cc)
	/tmp/workdir/go/src/runtime/sema.go:56 +0x2e
sync.(*WaitGroup).Wait(0x1868a0c0)
	/tmp/workdir/go/src/sync/waitgroup.go:131 +0x8d
sync_test.TestWaitGroupMisuse3(0x18698000)
	/tmp/workdir/go/src/sync/waitgroup_test.go:165 +0x136
testing.tRunner(0x18698000, 0x8161da4)
	/tmp/workdir/go/src/testing/testing.go:659 +0x79
created by testing.(*T).Run
	/tmp/workdir/go/src/testing/testing.go:701 +0x255
FAIL	sync	120.018s
2017/03/09 18:20:59 Failed: exit status 1
2017/03/09 18:20:59 FAILED

@bradfitz bradfitz added the Testing An issue that has been verified to require only test changes, not just a test failure. label Mar 21, 2017
@bradfitz bradfitz added this to the Go1.9Maybe milestone Mar 21, 2017
@ianlancetaylor ianlancetaylor added the NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. label Aug 3, 2017
@ianlancetaylor ianlancetaylor modified the milestones: Go1.10, Go1.9Maybe Aug 3, 2017
@rsc rsc closed this as completed Nov 29, 2017
@golang golang locked and limited conversation to collaborators Nov 29, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
FrozenDueToAge help wanted NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. 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

5 participants