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

bump libnetwork to 19279f0492417475b6bfbd0aa529f73e8f178fb5 #37252

Merged
merged 1 commit into from Jun 11, 2018

Conversation

thaJeztah
Copy link
Member

full diff: moby/libnetwork@3931ba4...19279f0

includes;

includes;

- moby/libnetwork#2178 Fix possible race on ingress programming
- moby/libnetwork#2180 Fix spurious deadlock in overlay driver

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah
Copy link
Member Author

ping @fcrisciani @ctelfer PTAL

Copy link
Contributor

@fcrisciani fcrisciani left a comment

Choose a reason for hiding this comment

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

LGTM

@codecov
Copy link

codecov bot commented Jun 9, 2018

Codecov Report

❗ No coverage uploaded for pull request base (master@5e11f66). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #37252   +/-   ##
=========================================
  Coverage          ?   34.96%           
=========================================
  Files             ?      609           
  Lines             ?    45019           
  Branches          ?        0           
=========================================
  Hits              ?    15740           
  Misses            ?    27178           
  Partials          ?     2101

Copy link
Member

@vdemeester vdemeester left a comment

Choose a reason for hiding this comment

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

LGTM 🐯

@vdemeester vdemeester merged commit d812dee into moby:master Jun 11, 2018
@thaJeztah thaJeztah deleted the bump-libnetwork branch June 11, 2018 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants