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

Add a new firewall ingress-policy "isolated" #1140

Merged

Conversation

swagatbora90
Copy link
Contributor

IngressPolicyIsolated ("isolated") behaves similar to ingress policy "same-bridge" with the exception that connections from the same bridge are also blocked. This is meant to be functionally equivalent to Docker network option "enable_icc" when set to false.

Resolves: #1107

@swagatbora90 swagatbora90 force-pushed the ingress-same-bridge-isolate branch 3 times, most recently from 9171d76 to 4a0cb70 Compare January 15, 2025 18:29
@swagatbora90 swagatbora90 marked this pull request as ready for review January 15, 2025 21:52
IngressPolicyIsolated ("isolated") behaves  similar to ingress policy "same-bridge" with the exception
that connections from the same bridge are also blocked. This is meant to be functionally equivalent
to Docker network option "enable_icc" when set to false.

Signed-off-by: Swagat Bora <[email protected]>
@swagatbora90 swagatbora90 force-pushed the ingress-same-bridge-isolate branch from 4a0cb70 to 96483a9 Compare January 27, 2025 21:24
@swagatbora90
Copy link
Contributor Author

PTAL @squeed @henry118

@squeed squeed self-requested a review March 3, 2025 16:26
Copy link
Member

@squeed squeed left a comment

Choose a reason for hiding this comment

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

Looks good! Can you please document this on https://github.com/containernetworking/cni.dev ? Thanks!

@squeed squeed merged commit 3c224f5 into containernetworking:main Mar 3, 2025
6 checks passed
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.

Does bridge plugin support disabling inter container connectivity?
2 participants