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

sharness: make t0180-p2p less racy #4310

Merged
merged 1 commit into from
Oct 17, 2017
Merged

sharness: make t0180-p2p less racy #4310

merged 1 commit into from
Oct 17, 2017

Conversation

magik6k
Copy link
Member

@magik6k magik6k commented Oct 16, 2017

Hopefully fixes #4282

This should make the test way more reliable, if it doesn't I'll look into it more.

License: MIT
Signed-off-by: Łukasz Magiera <[email protected]>
@ghost ghost assigned magik6k Oct 16, 2017
@ghost ghost added the status/in-progress In progress label Oct 16, 2017
Copy link
Member

@Stebalien Stebalien left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks for fixing this!

@whyrusleeping whyrusleeping merged commit c9705c6 into master Oct 17, 2017
@whyrusleeping whyrusleeping deleted the fix/p2p-sharness branch October 17, 2017 12:07
@ghost ghost removed the status/in-progress In progress label Oct 17, 2017
Copy link
Member

@Kubuxu Kubuxu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

much batter. I think I've let it pass last time. I will look our for things like this in future

@magik6k magik6k restored the fix/p2p-sharness branch November 27, 2017 03:34
@magik6k magik6k deleted the fix/p2p-sharness branch November 27, 2017 03:49
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

Successfully merging this pull request may close these issues.

t0180-p2p is racy
4 participants