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
Bug 1798788: Set Kubelet node IP to non-vip #1444
Bug 1798788: Set Kubelet node IP to non-vip #1444
Conversation
@celebdor: This pull request references Bugzilla bug 1797655, which is invalid:
Comment In response to this:
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. |
@celebdor: No Bugzilla bug is referenced in the title of this pull request. In response to this:
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. |
/bugzilla refresh |
@celebdor: No Bugzilla bug is referenced in the title of this pull request. In response to this:
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. |
@celebdor: This pull request references Bugzilla bug 1798788, which is invalid:
Comment In response to this:
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. |
I propose the new non_virtual_ip for all of the platforms. |
Only ran the scripts in the PR in isolation still, I have not deployed with a new MCO yet (it's 2:36am already). |
6af90ff
to
eaa8c95
Compare
/test e2e-ovirt |
eaa8c95
to
7a922d6
Compare
/test e2e-openstack |
7a922d6
to
a8dc8fa
Compare
/test e2e-ovirt |
This didn't work for me in my first run of this with a backport to 4.3. The new
|
@russellb: This pull request references Bugzilla bug 1798788, which is valid. In response to this:
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. |
This change prevents: * Kubelet choosing as node IP an address from a non control plane subnet (like the provisioning network) * Kubelet choosing a deprecated IPv6 address as its node IP that in several platform can be a VIP Signed-off-by: Antoni Segura Puimedon <antoni@redhat.com>
CRI-O on startup could get addresses with expiring leases or from interfaces that are in isolated networks. This patch extends kubelet-nodeip.service to fix CRI-O as well. Signed-off-by: Antoni Segura Puimedon <antoni@redhat.com>
2d40474
to
065082e
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: celebdor, kikisdeliveryservice, russellb 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 |
/retest Please review the full test history for this PR and help us cut down flakes. |
@celebdor: All pull requests linked via external trackers have merged. Bugzilla bug 1798788 has been moved to the MODIFIED state. In response to this:
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. |
@celebdor: The following tests failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. 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. I understand the commands that are listed here. |
/cherry-pick release-4.3 |
@russellb: #1444 failed to apply on top of branch "release-4.3":
In response to this:
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. |
opened #1466 |
This change prevents: * Kubelet choosing as node IP an address from a non control plane subnet (like the provisioning network) * Kubelet choosing a deprecated IPv6 address as its node IP that in several platform can be a VIP This ports the BM change from openshift#1444 to OpenStack platform.
We've updated MCO to ensure IPv6 VIPs are not accidentally used by kubelet. Make use of that value and set it as the address kube-apiserver should advertise. Otherwise, it uses the same broken default behavior that kubelet does and will incorrectly choose IPv6 VIPs, breaking the cluster. The change to specify the correct IP to kubelet is: openshift/machine-config-operator#1444
We've updated MCO to ensure IPv6 VIPs are not accidentally used by kubelet. Make use of that value and set it as the address kube-apiserver should advertise. Otherwise, it uses the same broken default behavior that kubelet does and will incorrectly choose IPv6 VIPs, breaking the cluster. The change to specify the correct IP to kubelet is: openshift/machine-config-operator#1444
We've updated MCO to ensure IPv6 VIPs are not accidentally used by kubelet. Make use of that value and set it as the address kube-apiserver should advertise. Otherwise, it uses the same broken default behavior that kubelet does and will incorrectly choose IPv6 VIPs, breaking the cluster. The change to specify the correct IP to kubelet is: openshift/machine-config-operator#1444
This change prevents: * Kubelet choosing as node IP an address from a non control plane subnet (like the provisioning network) * Kubelet choosing a deprecated IPv6 address as its node IP that in several platform can be a VIP This ports the BM change from openshift#1444 to OpenStack platform.
This change prevents:
(like the provisioning network)
several platform can be a VIP
- How to verify it
Deploy on an IPv6 environment on a platform that uses keepalived managed Virtual IPs for its API, DNS and Ingress and see that the node IP is none of those, as those are created by keepalived as deprecated addresses.