Bug 1957015: [on-prem] Backport duplicate VIP fixes#2566
Bug 1957015: [on-prem] Backport duplicate VIP fixes#2566openshift-merge-robot merged 2 commits intoopenshift:release-4.7from
Conversation
Due to a bug in keepalived 2.0.10, if the liveness probe kills keepalived, any vips that were assigned to the system remain and are not cleaned up when keepalived restarts. Until we get a fixed version of keepalived, let's clean up the VIPs ourselves before starting keepalived. If the node is supposed to have the VIP it will configure it again when keepalived starts.
openshift-ci-robot
added
bugzilla/severity-urgent
|
@cybertron: This pull request references Bugzilla bug 1957015, which is valid. The bug has been moved to the POST state. The bug has been updated to refer to the pull request using the external bug tracker. 6 validation(s) were run on this bug
No GitHub users were found matching the public email listed for the QA contact in Bugzilla (vvoronko@redhat.com), skipping review request. DetailsIn 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. |
|
/test e2e-openstack |
openshift-ci-robot
requested review from
Gal-Zaidman,
jcpowermac,
mandre,
patrickdillon and
yboaron
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: cybertron, mandre The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
/retest |
|
/retest All of the platforms that actually use this passed, and none of the failing jobs are technically required. Would be nice to have a little less red though... |
|
@kikisdeliveryservice can you approve this backport please? You approved the two original fixes to mainline. |
|
/approve |
openshift-ci
bot
added
the
approved
|
/retest Please review the full test history for this PR and help us cut down flakes. |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cybertron, JAORMX, mandre, yuqi-zhang The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
knobunc
added
the
cherry-pick-approved
|
@cybertron: The following tests failed, say
Full PR test history. Your PR dashboard. DetailsInstructions 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. |
|
@cybertron: All pull requests linked via external trackers have merged: Bugzilla bug 1957015 has been moved to the MODIFIED state. DetailsIn 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. |
9 participants
- What I did
This is a combined backport of the fixes in #2511 and #2548 . It works around a bug in keepalived that could cause duplicate VIPs to be assigned to nodes.
- How to verify it
Deploy a cluster. When this bug occurs, operators like console and authentication fail to deploy because they can't talk to the ingress VIP correctly.
- Description for the changelog
Fix an intermittent issue with VIPs being duplicated across multiple nodes.