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

TestReap_ReapChildren fails randomly on x86_32 (unreliable test?) #5

Closed
onlyjob opened this issue Aug 26, 2016 · 4 comments
Closed

Comments

@onlyjob
Copy link

onlyjob commented Aug 26, 2016

TestReap_ReapChildren usually fails on i686 as follows:

--- FAIL: TestReap_ReapChildren (1.00s) 
        reap_unix_test.go:53: should have reaped 18206 
@onlyjob
Copy link
Author

onlyjob commented Sep 1, 2016

ping?

@tpot
Copy link

tpot commented Sep 1, 2016

I'm pretty sure this is a race between the goroutine that calls ReapChildren() and hitting the select statement that pulls off the pids channel. I added a sleep after the goroutine to give it time to get started and the test hasn't failed in over an hour.

There's probably a nicer way to synchronise this though.

reap_unix_test_patch.txt

@slackpad
Copy link
Contributor

slackpad commented Sep 1, 2016

Thanks @tpot I went ahead and took your patch via 04ce4d0. This test is a bit flaky because it does have some timeouts, so hopefully this will be good enough. If it continues to fail randomly we can look into a better synchronization mechanism.

@slackpad slackpad closed this as completed Sep 1, 2016
@tpot
Copy link

tpot commented Sep 2, 2016

Thanks @slackpad!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants