Bug 2038252: Backport 4.9 OVN drop icmp frag from other nodes on Azure cluster #1273

mccv1r0 · 2022-01-07T16:43:24Z

Add logic to OVN-Kubernetes to determine what platform we are running on.

Only use drop icmp frag needed daemonset when on Azure platform
drop icmp frag needed received from other nodes in the cluster

Signed-off-by: Michael Cambria mcambria@redhat.com
(cherry picked from commit 7a05ada)

openshift-ci · 2022-01-07T16:43:28Z

@mccv1r0: This pull request references Bugzilla bug 2038252, which is invalid:

expected the bug to target the "4.9.z" release, but it targets "4.10.0" instead
expected Bugzilla bug 2038252 to depend on a bug targeting a release in 4.10.0 and in one of the following states: VERIFIED, RELEASE_PENDING, CLOSED (ERRATA), CLOSED (CURRENTRELEASE), but no dependents were found

Comment /bugzilla refresh to re-evaluate validity if changes to the Bugzilla bug are made, or edit the title of this pull request to link to a different bug.

In response to this:

Bug 2038252: Backport 4.9 OVN drop icmp frag from other nodes on Azure cluster

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.

mccv1r0 · 2022-01-07T16:45:21Z

/bugzilla refresh

openshift-ci · 2022-01-07T16:45:23Z

@mccv1r0: This pull request references Bugzilla bug 2038252, which is invalid:

expected Bugzilla bug 2038252 to depend on a bug targeting a release in 4.10.0 and in one of the following states: VERIFIED, RELEASE_PENDING, CLOSED (ERRATA), CLOSED (CURRENTRELEASE), but no dependents were found

Comment /bugzilla refresh to re-evaluate validity if changes to the Bugzilla bug are made, or edit the title of this pull request to link to a different bug.

In response to this:

/bugzilla refresh

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.

mccv1r0 · 2022-01-07T16:48:25Z

/bugzilla refresh

openshift-ci · 2022-01-07T16:48:27Z

@mccv1r0: This pull request references Bugzilla bug 2038252, which is invalid:

expected Bugzilla bug 2038252 to depend on a bug targeting a release in 4.10.0 and in one of the following states: VERIFIED, RELEASE_PENDING, CLOSED (ERRATA), CLOSED (CURRENTRELEASE), but no dependents were found

Comment /bugzilla refresh to re-evaluate validity if changes to the Bugzilla bug are made, or edit the title of this pull request to link to a different bug.

In response to this:

/bugzilla refresh

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.

mccv1r0 · 2022-01-07T16:51:01Z

/bugzilla refresh

openshift-ci · 2022-01-07T16:51:04Z

@mccv1r0: This pull request references Bugzilla bug 2038252, which is invalid:

expected Bugzilla bug 2038252 to depend on a bug targeting a release in 4.10.0 and in one of the following states: VERIFIED, RELEASE_PENDING, CLOSED (ERRATA), CLOSED (CURRENTRELEASE), but no dependents were found

Comment /bugzilla refresh to re-evaluate validity if changes to the Bugzilla bug are made, or edit the title of this pull request to link to a different bug.

In response to this:

/bugzilla refresh

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.

…g on. Only use drop icmp frag needed daemonset when on Azure platform drop icmp frag needed received from other nodes in the cluster Signed-off-by: Michael Cambria <mcambria@redhat.com> (cherry picked from commit 7a05ada)

mccv1r0 · 2022-01-07T17:21:37Z

/bugzilla refresh

openshift-ci · 2022-01-07T17:21:45Z

@mccv1r0: This pull request references Bugzilla bug 2038252, 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

bug is open, matching expected state (open)
bug target release (4.9.z) matches configured target release for branch (4.9.z)
bug is in the state ASSIGNED, which is one of the valid states (NEW, ASSIGNED, ON_DEV, POST, POST)
dependent bug Bugzilla bug 1988483 is in the state VERIFIED, which is one of the valid states (VERIFIED, RELEASE_PENDING, CLOSED (ERRATA), CLOSED (CURRENTRELEASE))
dependent Bugzilla bug 1988483 targets the "4.10.0" release, which is one of the valid target releases: 4.10.0
bug has dependents

Requesting review from QA contact:
/cc @anuragthehatter

In response to this:

/bugzilla refresh

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.

mccv1r0 · 2022-01-07T20:04:38Z

/retest-required

mccv1r0 · 2022-01-07T20:33:47Z

/retest-required

mccv1r0 · 2022-01-10T14:38:57Z

/retest-required

mccv1r0 · 2022-01-10T18:40:22Z

/retest-required

mccv1r0 · 2022-01-10T20:27:00Z

/retest-required

mccv1r0 · 2022-01-11T15:59:13Z

Waiting on: #1270

trozet · 2022-01-12T19:45:53Z

/lgtm
/label backport-risk-assessed

openshift-ci · 2022-01-12T19:46:30Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mccv1r0, trozet

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:

~~OWNERS~~ [trozet]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

openshift-bot · 2022-01-24T10:40:08Z

/retest-required

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2022-01-24T11:52:10Z

/retest-required

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2022-01-24T12:16:09Z

/retest-required

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2022-01-24T12:28:19Z

/retest-required

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2022-01-24T12:52:10Z

/retest-required

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2022-01-24T13:04:11Z

/retest-required

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2022-01-24T13:28:08Z

/retest-required

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2022-01-24T13:40:09Z

/retest-required

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2022-01-24T14:16:11Z

/retest-required

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2022-01-24T14:28:10Z

/retest-required

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2022-01-24T16:22:09Z

/retest-required

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2022-01-24T18:59:09Z

/retest-required

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2022-01-24T19:38:10Z

/retest-required

Please review the full test history for this PR and help us cut down flakes.

danwinship · 2022-01-24T19:40:46Z

/override ci/prow/e2e-metal-ipi-ovn-ipv6-ipsec
/override ci/prow/e2e-gcp-ovn-upgrade
/override ci/prow/e2e-azure-ovn

openshift-ci · 2022-01-24T19:49:55Z

@danwinship: Overrode contexts on behalf of danwinship: ci/prow/e2e-azure-ovn, ci/prow/e2e-gcp-ovn-upgrade, ci/prow/e2e-metal-ipi-ovn-ipv6-ipsec

In response to this:

/override ci/prow/e2e-metal-ipi-ovn-ipv6-ipsec
/override ci/prow/e2e-gcp-ovn-upgrade
/override ci/prow/e2e-azure-ovn

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.

anuragthehatter · 2022-02-08T22:27:08Z

/label qe-approved

anuragthehatter · 2022-02-08T22:27:21Z

/label cherry-pick-approved

openshift-bot · 2022-02-08T23:22:59Z

/retest-required

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2022-02-09T00:34:56Z

/retest-required

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2022-02-09T01:47:00Z

/retest-required

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2022-02-09T03:02:42Z

/retest-required

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2022-02-09T07:59:44Z

/retest-required

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2022-02-09T08:59:45Z

/retest-required

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2022-02-09T11:35:46Z

/retest-required

Please review the full test history for this PR and help us cut down flakes.

openshift-bot · 2022-02-09T13:11:59Z

/retest-required

Please review the full test history for this PR and help us cut down flakes.

openshift-ci · 2022-02-09T13:55:21Z

@mccv1r0: all tests passed!

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.

openshift-ci · 2022-02-09T13:58:19Z

@mccv1r0: All pull requests linked via external trackers have merged:

openshift/cluster-network-operator#1273

Bugzilla bug 2038252 has been moved to the MODIFIED state.

In response to this:

Bug 2038252: Backport 4.9 OVN drop icmp frag from other nodes on Azure cluster

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.

openshift-ci bot added bugzilla/severity-urgent Referenced Bugzilla bug's severity is urgent for the branch this PR is targeting. bugzilla/invalid-bug Indicates that a referenced Bugzilla bug is invalid for the branch this PR is targeting. labels Jan 7, 2022

openshift-ci bot requested review from aojea and danwinship January 7, 2022 16:43

mccv1r0 force-pushed the azure-ovndrop-49 branch from 159e7b7 to bcaff61 Compare January 7, 2022 17:08

openshift-ci bot added bugzilla/valid-bug Indicates that a referenced Bugzilla bug is valid for the branch this PR is targeting. and removed bugzilla/invalid-bug Indicates that a referenced Bugzilla bug is invalid for the branch this PR is targeting. labels Jan 7, 2022

openshift-ci bot requested a review from anuragthehatter January 7, 2022 17:21

openshift-ci bot added the backport-risk-assessed Indicates a PR to a release branch has been evaluated and considered safe to accept. label Jan 12, 2022

openshift-ci bot assigned trozet Jan 12, 2022

openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jan 12, 2022

openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 12, 2022

openshift-ci bot added the qe-approved Signifies that QE has signed off on this PR label Feb 8, 2022

openshift-ci bot added the cherry-pick-approved Indicates a cherry-pick PR into a release branch has been approved by the release branch manager. label Feb 8, 2022

openshift-merge-robot merged commit cde9541 into openshift:release-4.9 Feb 9, 2022

Bug 2038252: Backport 4.9 OVN drop icmp frag from other nodes on Azure cluster #1273

Bug 2038252: Backport 4.9 OVN drop icmp frag from other nodes on Azure cluster #1273

Conversation

mccv1r0 commented Jan 7, 2022

openshift-ci bot commented Jan 7, 2022

mccv1r0 commented Jan 7, 2022

openshift-ci bot commented Jan 7, 2022

mccv1r0 commented Jan 7, 2022

openshift-ci bot commented Jan 7, 2022

mccv1r0 commented Jan 7, 2022

openshift-ci bot commented Jan 7, 2022

mccv1r0 commented Jan 7, 2022

openshift-ci bot commented Jan 7, 2022

mccv1r0 commented Jan 7, 2022

mccv1r0 commented Jan 7, 2022

mccv1r0 commented Jan 10, 2022

mccv1r0 commented Jan 10, 2022

mccv1r0 commented Jan 10, 2022

mccv1r0 commented Jan 11, 2022

trozet commented Jan 12, 2022

openshift-ci bot commented Jan 12, 2022

openshift-bot commented Jan 24, 2022

openshift-bot commented Jan 24, 2022

openshift-bot commented Jan 24, 2022

openshift-bot commented Jan 24, 2022

openshift-bot commented Jan 24, 2022

openshift-bot commented Jan 24, 2022

openshift-bot commented Jan 24, 2022

openshift-bot commented Jan 24, 2022

openshift-bot commented Jan 24, 2022

openshift-bot commented Jan 24, 2022

openshift-bot commented Jan 24, 2022

openshift-bot commented Jan 24, 2022

openshift-bot commented Jan 24, 2022

danwinship commented Jan 24, 2022

openshift-ci bot commented Jan 24, 2022

anuragthehatter commented Feb 8, 2022

anuragthehatter commented Feb 8, 2022

openshift-bot commented Feb 8, 2022

openshift-bot commented Feb 9, 2022

openshift-bot commented Feb 9, 2022

openshift-bot commented Feb 9, 2022

openshift-bot commented Feb 9, 2022

openshift-bot commented Feb 9, 2022

openshift-bot commented Feb 9, 2022

openshift-bot commented Feb 9, 2022

openshift-ci bot commented Feb 9, 2022

openshift-ci bot commented Feb 9, 2022