-
Notifications
You must be signed in to change notification settings - Fork 116
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix listener locking #147
fix listener locking #147
Conversation
69698d1
to
e31debd
Compare
Just a set of "standard" builds: @dhood Maybe you can trigger some builds which cover some specific tests previously flaky with a larger number of test repetitions. |
CI retesting only |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added dd693c3 to aid future debugging
Nice! Thanks. |
@dhood and I have been looking at the deadlocks some more together and finally figured out that #132 (comment) introduced a regression (more details there).
Hopefully this patch will cover #142 as well a #143.
馃憦 goes to @dhood for figuring out all these cases and working on the fixes.