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

Update vendored docker and libnetwork #1684

Merged
merged 1 commit into from
Oct 24, 2016

Conversation

aaronlehmann
Copy link
Collaborator

@aaronlehmann aaronlehmann commented Oct 24, 2016

This brings in a race fix from docker/docker. It also updates libnetwork
to keep it in sync with changes in the docker/docker code.

Finally, logrus to use the same fork that docker uses now.

This brings in a race fix from docker/docker. It also updates libnetwork
to keep it in sync with changes in the docker/docker code.

Finally, logrus to use the same fork that docker uses now.

Signed-off-by: Aaron Lehmann <[email protected]>
@aaronlehmann
Copy link
Collaborator Author

Also sync'd logrus with docker master.

@codecov-io
Copy link

codecov-io commented Oct 24, 2016

Current coverage is 56.38% (diff: 0.00%)

Merging #1684 into master will increase coverage by 0.20%

@@             master      #1684   diff @@
==========================================
  Files            89         89          
  Lines         14393      14407    +14   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits           8086       8124    +38   
+ Misses         5235       5226     -9   
+ Partials       1072       1057    -15   

Sunburst

Powered by Codecov. Last update bcedc9f...c403f2a

@aaronlehmann
Copy link
Collaborator Author

Integration test failure is affecting multiple PRs. It's tracked here: #1680.

@aluzzardi
Copy link
Member

LGTM

@aluzzardi aluzzardi merged commit dd27565 into moby:master Oct 24, 2016
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.

4 participants