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

Docker compose labels #8581

Closed
wants to merge 6 commits into from
Closed

Conversation

yuyangbj
Copy link
Contributor

Fixes #6405
[specific ci=3-01-Docker-Compose]

After make a new network bridge network, the port mapping for containers which
uses this new network will be lost after the VCH is rebooted from VC. And the
iptables for the port mapping in VCH are also lost.
With traditional docker, the network created by docker-compose
contains labels property with empty object {}. But in VIC, labels
will be set to NULL. This error is only found when docker-compose
version is 2.0.
@yuyangbj yuyangbj requested a review from a team as a code owner August 29, 2019 05:44
@codecov-io
Copy link

codecov-io commented Aug 29, 2019

Codecov Report

Merging #8581 into master will decrease coverage by 0.02%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #8581      +/-   ##
==========================================
- Coverage    31.4%   31.37%   -0.03%     
==========================================
  Files         293      293              
  Lines       44376    44379       +3     
==========================================
- Hits        13937    13926      -11     
- Misses      29220    29235      +15     
+ Partials     1219     1218       -1
Impacted Files Coverage Δ
lib/apiservers/engine/backends/network.go 0% <0%> (ø) ⬆️
pkg/logmgr/logmgr.go 62.79% <0%> (-3.88%) ⬇️
lib/portlayer/attach/communication/interactor.go 24.08% <0%> (-2.92%) ⬇️
lib/portlayer/event/subscriber.go 92.2% <0%> (-2.6%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 19d8a93...1dedfc2. Read the comment docs.

@yuyangbj yuyangbj closed this Aug 29, 2019
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.

docker-compose up -d fails when network already exists.
3 participants