This repository has been archived by the owner on Jun 20, 2024. It is now read-only.
Forbid traffic to the Weave port from containers #1670
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
inercia
force-pushed
the
issues/1634-block-6783-6784
branch
from
524154a to
0d76817
Compare
|
Please rebase so the tests run against Docker 1.9 |
Blocked by #1675 |
inercia
force-pushed
the
issues/1634-block-6783-6784
branch
from
0d76817 to
b48058c
Compare
|
Apologies, the original issue was a bit muddled - it stated we needed to block UDP 6783/6784 but we also need to block TCP 6783 too (I've updated the subject to reflect this). This PR blocks TCP & UDP 6783 - can we block |
|
@awh So, just to be sure we are on the same page, we need to block |
Make sure we have the Docker bridge IP before adding/removing iptables rules
inercia
force-pushed
the
issues/1634-block-6783-6784
branch
from
b48058c to
ddb390d
Compare
awh
added a commit
that referenced
this pull request
Nov 26, 2015
Forbid traffic to the Weave port from containers
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #1634