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
[release-4.13] OCPBUGS-17123: Remove stale egressip status entry #1793
[release-4.13] OCPBUGS-17123: Remove stale egressip status entry #1793
Conversation
This commit ensures egress ip status is in sync with cloud private ip config on a restarted ovnk master because stale entry is noticed in the egress ip status though cloud private ip config entry is not present for the given egress ip. Signed-off-by: Periyasamy Palanisamy <pepalani@redhat.com>
This would remove node name comparison between cloud private ip config and egress ip status as this is not valid in the case of egress ip failover to another node during upgrade or reboot scenario. Signed-off-by: Periyasamy Palanisamy <pepalani@redhat.com>
@pperiyasamy: This pull request references Jira Issue OCPBUGS-17123, which is invalid:
Comment The bug has been updated to refer to the pull request using the external bug tracker. 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. |
/assign @jcaamano @flavio-fernandes @tssurya |
/jira refresh |
@pperiyasamy: This pull request references Jira Issue OCPBUGS-17123, which is valid. The bug has been moved to the POST state. 6 validation(s) were run on this bug
No GitHub users were found matching the public email listed for the QA contact in Jira (jechen@redhat.com), skipping review 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. |
/retest-required |
/lgtm TY Peri! |
/ocpbugs cc-qa |
/label qe-approved |
/retest-required |
/retest |
/test e2e-openstack-ovn |
1 similar comment
/test e2e-openstack-ovn |
/approve |
/label backport-risk-assessed |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: flavio-fernandes, jcaamano, pperiyasamy 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 |
/label cherry-pick-approved |
/test e2e-openstack-ovn |
/test e2e-aws-ovn-shared-to-local-gateway-mode-migration |
/test e2e-aws-ovn-local-to-shared-gateway-mode-migration |
/test e2e-aws-ovn-shared-to-local-gateway-mode-migration |
/test e2e-openstack-ovn |
@pperiyasamy: The following test failed, say
Full PR test history. Your PR dashboard. 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. |
16df936
into
openshift:release-4.13
@pperiyasamy: Jira Issue OCPBUGS-17123: All pull requests linked via external trackers have merged: Jira Issue OCPBUGS-17123 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. |
Manual cherry-pick of 152ae1e 8dd5d2e c08382b and 712978d.
Conflicts due to complete redesign of egress ip module.
Needed to squash first 3 commits manually and did manual commit for the last one.