Skip to content

Add Pod lifecycle to remove redirect iptables rule#290

Open
brunopaiuca wants to merge 1 commit into
jtblin:mainfrom
brunopaiuca:implement-graceful-iptables-rule-removal
Open

Add Pod lifecycle to remove redirect iptables rule#290
brunopaiuca wants to merge 1 commit into
jtblin:mainfrom
brunopaiuca:implement-graceful-iptables-rule-removal

Conversation

@brunopaiuca

Copy link
Copy Markdown

Implement Pod lifecycle to remove redirect iptables rules to avoid
traffic be redirected to one port service that is not working anymore.

It is implemented using Pod Lifecycle preStop to remove the rule when
the kube2iam DaemonSet is deleted or the pod die somehow.

Implement Pod lifecycle to remove redirect iptables rules to avoid
traffic be redirected to one port service that is not working anymore.

It is implemented using Pod Lifecycle preStop to remove the rule when
the kube2iam DaemonSet is deleted or the pod die somehow.
@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 19.481% when pulling f21e1c8 on brunopaiuca:implement-graceful-iptables-rule-removal into a1b6aa3 on jtblin:master.

Comment thread README.md
* for [OpenShift](https://www.openshift.org/) use `tun0`
* for [Cilium](https://www.cilium.io) use `lxc+`

**Warning**: It is important ensure the iptables rule will be deleted when you delete the DaemonSet. Below is the command proposal to deal with it, we recommend to use the comand in the Pod Lifecycle:

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

typo: s/comand/command/

@jtblin

jtblin commented Dec 21, 2020

Copy link
Copy Markdown
Owner

cc @mariusv @walkafwalka can you help reviewing helm charts changes pls?

@mariusv mariusv left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please bump the chart version to the next minor release as this is a breaking change

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