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

[1.12.6] BACKPORT: Handle containers with different network drivers #243

Merged
merged 1 commit into from
May 23, 2017

Conversation

runcom
Copy link
Collaborator

@runcom runcom commented May 1, 2017

Upstream reference: moby/libnetwork#1480
Fix BZ#1446526

FIx Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1446526

There is no guarantees that the ep and extEp are using the same driver.
If they are not using the same drivers, the driver for ep will not know
about the networks of extEp and fails the RevokeExternalConnectivity
call.

Signed-off-by: Shayan Pooya [email protected]
Signed-off-by: Antonio Murdaca [email protected]

Upstream reference: moby/libnetwork#1480
Fix BZ#1446526

There is no guarantees that the ep and extEp are using the same driver.
If they are not using the same drivers, the driver for ep will not know
about the networks of extEp and fails the RevokeExternalConnectivity
call.

Signed-off-by: Shayan Pooya <[email protected]>
Signed-off-by: Antonio Murdaca <[email protected]>
@rh-atomic-bot
Copy link

@runcom runcom changed the title BACKPORT: Handle containers with different network drivers [1.12.6] BACKPORT: Handle containers with different network drivers May 1, 2017
@runcom
Copy link
Collaborator Author

runcom commented May 3, 2017

@rhatdan PTAL

@rhatdan
Copy link
Member

rhatdan commented May 23, 2017

LGTM

@rhatdan rhatdan merged commit 94ed431 into projectatomic:docker-1.12.6 May 23, 2017
@runcom runcom deleted the fix-bz-1446526 branch May 23, 2017 15:52
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.

4 participants