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
kube-proxy react on Node PodCIDR changes #113252
kube-proxy react on Node PodCIDR changes #113252
Conversation
Hi @code-elinka. Thanks for your PR. I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@code-elinka: This issue is currently awaiting triage. If a SIG or subproject determines this is a relevant issue, they will accept it by applying the The Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/ok-to-test |
this commit should not be here service update event should be triggered when appProtocol in port is … ddc62aa3682d931b0a6473c034541f32d583c112 /hold |
8072582
to
9dc65ac
Compare
9dc65ac
to
24f1e91
Compare
Kube/proxy, in NodeCIDR local detector mode, uses the node.Spec.PodCIDRs field to build the Services iptables rules. The Node object depends on the kubelet, but if kube-proxy runs as a static pods or as a standalone binary, it is not possible to guarantee that the values obtained at bootsrap are valid, causing traffic outages. Kube-proxy has to react on node changes to avoid this problems, it simply restarts if detect that the node PodCIDRs have changed. In case that the Node has been deleted, kube-proxy will only log an error and keep working, since it may break graceful shutdowns of the node.
24f1e91
to
50c6f14
Compare
Done, I dropped unrelated changes. |
/retitle kube-proxy react on Node PodCIDR changes |
/lgtm |
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.
/lgtm
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: aojea, code-elinka, xmudrii The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
…#113252-upstream-release-1.23 Automated cherry pick of #113252: kube-proxy: gate topology correctly
/retest |
My suggestion for the changelog text: `kube-proxy` will now restart if it detects changes to the `.spec.podCIDRs` field of its node What do we mean by “restart”? Do we mean “exit”, and then the restart is implicit? |
yeah, it will really exit, the restart is the implicit, :/ |
Cherry pick of #113247 on release-1.24.
#113247: service update event should be triggered when appProtocol
For details on the cherry pick process, see the cherry pick requests page.