Bug 1823557: backport fixes to support upstream->downstream merges by dcbw · Pull Request #589 · openshift/cluster-network-operator

dcbw · 2020-04-13T16:33:08Z

Backports of #466, #467, and #572 . These are needed to make nbctl daemon mode work while at the same time being able to check local OVS database to figure out if SCTP is supported by OVN.

@trozet @knobunc @fepan @openshift/networking

openshift-ci-robot · 2020-04-13T16:33:10Z

@dcbw: No Bugzilla bug is referenced in the title of this pull request.
To reference a bug, add 'Bug XXX:' to the title of this pull request and request another bug refresh with /bugzilla refresh.

Details

In response to this:

Backport fixes to support upstream->downstream merges

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.

dcbw · 2020-04-13T16:33:21Z

/test e2e-aws-ovn

openshift-ci-robot · 2020-04-13T16:33:36Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dcbw

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

~~OWNERS~~ [dcbw]

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

openshift-ci-robot · 2020-04-13T16:46:55Z

@dcbw: No Bugzilla bug is referenced in the title of this pull request.
To reference a bug, add 'Bug XXX:' to the title of this pull request and request another bug refresh with /bugzilla refresh.

Details

In response to this:

Backport fixes to support upstream->downstream merges

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.

dcbw · 2020-04-13T20:49:02Z

/retest

openshift-ci-robot · 2020-04-13T21:57:06Z

@dcbw: This pull request references Bugzilla bug 1811811, which is invalid:

expected the bug to target the "4.4.0" release, but it targets "4.5.0" instead
expected Bugzilla bug 1811811 to depend on a bug targeting a release in 4.5.0 and in one of the following states: VERIFIED, RELEASE_PENDING, CLOSED (ERRATA), 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.

Details

In response to this:

Bug 1811811: backport fixes to support upstream->downstream merges

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-robot · 2020-04-13T21:59:29Z

@dcbw: This pull request references Bugzilla bug 1823557, which is invalid:

expected dependent Bugzilla bug 1718372 to be in one of the following states: VERIFIED, RELEASE_PENDING, CLOSED (ERRATA), but it is ON_QA instead
expected dependent Bugzilla bug 1811815 to be in one of the following states: VERIFIED, RELEASE_PENDING, CLOSED (ERRATA), but it is ON_QA instead
expected dependent Bugzilla bug 1718372 to target a release in 4.5.0, but it targets "---" instead

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.

Details

In response to this:

Bug 1823557: backport fixes to support upstream->downstream merges

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.

dcbw · 2020-04-13T22:01:43Z

/bugzilla refresh

openshift-ci-robot · 2020-04-13T22:01:46Z

@dcbw: This pull request references Bugzilla bug 1823557, which is invalid:

expected dependent Bugzilla bug 1718372 to be in one of the following states: VERIFIED, RELEASE_PENDING, CLOSED (ERRATA), but it is ON_QA instead
expected dependent Bugzilla bug 1811815 to be in one of the following states: VERIFIED, RELEASE_PENDING, CLOSED (ERRATA), but it is ON_QA instead
expected dependent Bugzilla bug 1718372 to target a release in 4.5.0, but it targets "---" instead

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.

Details

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.

dcbw · 2020-04-13T22:02:10Z

/bugzilla refresh

openshift-ci-robot · 2020-04-13T22:02:12Z

@dcbw: This pull request references Bugzilla bug 1823557, which is invalid:

expected dependent Bugzilla bug 1811815 to be in one of the following states: VERIFIED, RELEASE_PENDING, CLOSED (ERRATA), but it is ON_QA instead

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.

Details

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.

trozet · 2020-04-13T22:06:27Z

@dcbw I already have a bz and pr approved for 4.4 ssl fix

…cluster status (cherry picked from commit 31476cc)

Completes the switch from ovn2.11 to ovn2.12 by removing support for the now obsolete ovn2.11 SDN-643 https://issues.redhat.com/browse/SDN-643 Signed-off-by: Phil Cameron <pcameron@redhat.com> (cherry picked from commit f479a53)

dcbw · 2020-04-14T14:04:40Z

@trozet and I see that merged; which is great. So this is now just the other cleanups.

squeed · 2020-06-24T10:07:23Z

@dcbw do we still need this?

openshift-ci-robot · 2020-09-09T09:28:22Z

@dcbw: The following tests failed, say /retest to rerun all failed tests:

Test name	Commit	Details	Rerun command
ci/prow/e2e-gcp	`170f892`	link	`/test e2e-gcp`
ci/prow/e2e-gcp-upgrade	`170f892`	link	`/test e2e-gcp-upgrade`
ci/prow/images	`170f892`	link	`/test images`
ci/prow/verify	`170f892`	link	`/test verify`
ci/prow/unit	`170f892`	link	`/test unit`
ci/prow/e2e-gcp-ovn	`170f892`	link	`/test e2e-gcp-ovn`
ci/prow/e2e-upgrade	`170f892`	link	`/test e2e-upgrade`

Full PR test history. Your PR dashboard.

Details

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.

squeed · 2020-10-14T16:10:42Z

Closing this for now; @dcbw reopen if you like.

openshift-ci-robot requested review from pecameron and squeed April 13, 2020 16:33

openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Apr 13, 2020

dcbw changed the title ~~Backport fixes to support upstream->downstream merges~~ Bug 1811811: backport fixes to support upstream->downstream merges Apr 13, 2020

openshift-ci-robot added the bugzilla/invalid-bug Indicates that a referenced Bugzilla bug is invalid for the branch this PR is targeting. label Apr 13, 2020

dcbw changed the title ~~Bug 1811811: backport fixes to support upstream->downstream merges~~ Bug 1823557: backport fixes to support upstream->downstream merges Apr 13, 2020

dcbw mentioned this pull request Apr 13, 2020

Bug 1823560: backport fixes to support upstream->downstream merges #590

Closed

alexanderConstantinescu and others added 2 commits April 14, 2020 09:00

Changing OVN DB readinessProbe to check membership status instead of …

99ca3f6

…cluster status (cherry picked from commit 31476cc)

Remove support for ovn2.11

170f892

Completes the switch from ovn2.11 to ovn2.12 by removing support for the now obsolete ovn2.11 SDN-643 https://issues.redhat.com/browse/SDN-643 Signed-off-by: Phil Cameron <pcameron@redhat.com> (cherry picked from commit f479a53)

dcbw force-pushed the 44-support-upstream branch from fe4b958 to 170f892 Compare April 14, 2020 14:04

squeed closed this Oct 14, 2020

Conversation

dcbw commented Apr 13, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

openshift-ci-robot commented Apr 13, 2020

Uh oh!

dcbw commented Apr 13, 2020

Uh oh!

openshift-ci-robot commented Apr 13, 2020

Uh oh!

openshift-ci-robot commented Apr 13, 2020

Uh oh!

dcbw commented Apr 13, 2020

Uh oh!

openshift-ci-robot commented Apr 13, 2020

Uh oh!

openshift-ci-robot commented Apr 13, 2020

Uh oh!

dcbw commented Apr 13, 2020

Uh oh!

openshift-ci-robot commented Apr 13, 2020

Uh oh!

dcbw commented Apr 13, 2020

Uh oh!

openshift-ci-robot commented Apr 13, 2020

Uh oh!

trozet commented Apr 13, 2020

Uh oh!

dcbw commented Apr 14, 2020

Uh oh!

squeed commented Jun 24, 2020

Uh oh!

openshift-ci-robot commented Sep 9, 2020

Uh oh!

squeed commented Oct 14, 2020

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

dcbw commented Apr 13, 2020 •

edited

Loading