-
Notifications
You must be signed in to change notification settings - Fork 417
Switch to iptables-wrappers #73641
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
Switch to iptables-wrappers #73641
Conversation
sergiodj
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @Taffer. Small question about a change.
|
|
7dd5d6f to
ef6741f
Compare
121f7a1 to
b70c7f0
Compare
f76cf0a to
2db8520
Compare
This comment was marked as off-topic.
This comment was marked as off-topic.
EyeCantCU
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work on this! Documenting for anyone passing by - linkerd2 will need an iptables-wrappers-xtables-privileged packaged, or alternatively, just an xtables-privileged package will need to be created that only includes xtables and provides xtables (so that it can be used with iptables-wrappers). Removing iptables at runtime for now is the right call
This reverts commit f09afb0.
This reverts commit f09afb0.
Everything that can be using
iptables-wrappersshould be usingiptables-wrappers. These all built/tested locally ("works on my machine"), so 🤞 ...