Skip to content

OCPBUGS-16120: Dockerfile: bump to ovn 23.06#1719

Closed
dcbw wants to merge 1 commit intoopenshift:masterfrom
dcbw:ovn23.06
Closed

OCPBUGS-16120: Dockerfile: bump to ovn 23.06#1719
dcbw wants to merge 1 commit intoopenshift:masterfrom
dcbw:ovn23.06

Conversation

@dcbw
Copy link
Contributor

@dcbw dcbw commented Jun 22, 2023 •
edited
Loading 

New features and improvements:
- ARP proxy support (needed for CNV)
- Per-Load Balancer ct_flush
- Tiered ACLs
- northd: Update obsolete datapath groups instead of re-creating (perf/scale)

Relevant bug fixes:
- controller: Handle OpenFlow errors (rhbz#2134880)
- ovn-controller: fixed port not always set down when unbinding interface (rhbz#2150905)
- ovn-controller.c: Fix assertion failure during address set update

@openshift-ci openshift-ci bot requested review from abhat and trozet June 22, 2023 14:42
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 22, 2023

[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:

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

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 22, 2023
@dcbw
Copy link
Contributor Author

dcbw commented Jun 22, 2023

/retest

@dcbw
Copy link
Contributor Author

dcbw commented Jun 22, 2023

/test images

@dcbw
Copy link
Contributor Author

dcbw commented Jun 22, 2023

/retest

@dcbw
Copy link
Contributor Author

dcbw commented Jun 28, 2023

/retest-required

@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jun 30, 2023
@openshift-merge-robot openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jul 4, 2023
@dcbw
Copy link
Contributor Author

dcbw commented Jul 5, 2023

/retest

@dcbw
Dockerfile: bump to OVN 23.06
3d3c980 
New features and improvements:
- ARP proxy support (needed for CNV)
- Per-Load Balancer ct_flush
- Tiered ACLs
- northd: Update obsolete datapath groups instead of re-creating (perf/scale)

Relevant bug fixes:
- controller: Handle OpenFlow errors (rhbz#2134880)
- ovn-controller: fixed port not always set down when unbinding interface (rhbz#2150905)
- ovn-controller.c: Fix assertion failure during address set update
@dcbw
Copy link
Contributor Author

dcbw commented Jul 10, 2023

/retest
now that the OVNKube retry alert is fixed, and also for various AWS/EC2 install issues

@dcbw
Copy link
Contributor Author

dcbw commented Jul 10, 2023

/retest

level=error msg=Error: creating EC2 Instance: InsufficientInstanceCapacity: We currently do not have sufficient m6a.xlarge capacity in the Availability Zone you requested (us-east-1b). Our system will be working on provisioning additional capacity. You can currently get m6a.xlarge capacity by not specifying an Availability Zone in your request or choosing us-east-1a, us-east-1c, us-east-1d, us-east-1f.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jul 10, 2023 •
edited
Loading

@dcbw: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-openstack-ovn 3d3c980 link false /test e2e-openstack-ovn
ci/prow/e2e-ovn-hybrid-step-registry 3d3c980 link false /test e2e-ovn-hybrid-step-registry

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.

@dcbw
Copy link
Contributor Author

dcbw commented Jul 11, 2023

/retest-required
now that https://issues.redhat.com/browse/OCPBUGS-15992 is really fixed for metal-dualstack

@dcbw
Copy link
Contributor Author

dcbw commented Jul 12, 2023

/payload 4.14 nightly blocking

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jul 12, 2023

@dcbw: trigger 7 job(s) of type blocking for the nightly release of OCP 4.14

  • periodic-ci-openshift-release-master-nightly-4.14-e2e-aws-sdn-upgrade
  • periodic-ci-openshift-release-master-ci-4.14-e2e-azure-ovn-upgrade
  • periodic-ci-openshift-release-master-ci-4.14-upgrade-from-stable-4.13-e2e-gcp-ovn-rt-upgrade
  • periodic-ci-openshift-release-master-nightly-4.14-e2e-aws-ovn-serial
  • periodic-ci-openshift-release-master-ci-4.14-e2e-aws-ovn-upgrade
  • periodic-ci-openshift-release-master-nightly-4.14-e2e-metal-ipi-ovn-ipv6
  • periodic-ci-openshift-release-master-nightly-4.14-e2e-metal-ipi-sdn-bm

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/4ea47070-20d0-11ee-8797-b9048dcc20ff-0

@dcbw dcbw changed the title Dockerfile: bump to ovn 23.06 OCPBUGS-16120: Dockerfile: bump to ovn 23.06 Jul 12, 2023
@openshift-ci-robot openshift-ci-robot added jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. jira/valid-bug Indicates that a referenced Jira bug is valid for the branch this PR is targeting. labels Jul 12, 2023
@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Jul 12, 2023

@dcbw: This pull request references Jira Issue OCPBUGS-16120, which is valid. The bug has been moved to the POST state.

3 validation(s) were run on this bug
  • bug is open, matching expected state (open)
  • bug target version (4.14.0) matches configured target version for branch (4.14.0)
  • bug is in the state New, which is one of the valid states (NEW, ASSIGNED, POST)

Requesting review from QA contact:
/cc @anuragthehatter

The bug has been updated to refer to the pull request using the external bug tracker.

Details

In response to this:

New features and improvements:

  • ARP proxy support (needed for CNV)
  • Per-Load Balancer ct_flush
  • Tiered ACLs
  • northd: Update obsolete datapath groups instead of re-creating (perf/scale)

Relevant bug fixes:

  • controller: Handle OpenFlow errors (rhbz#2134880)
  • ovn-controller: fixed port not always set down when unbinding interface (rhbz#2150905)
  • ovn-controller.c: Fix assertion failure during address set update

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 openshift-ci bot requested a review from anuragthehatter July 12, 2023 16:26
@dcbw
Copy link
Contributor Author

dcbw commented Jul 13, 2023

/payload 4.14 nightly blocking

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jul 13, 2023

@dcbw: trigger 7 job(s) of type blocking for the nightly release of OCP 4.14

  • periodic-ci-openshift-release-master-nightly-4.14-e2e-aws-sdn-upgrade
  • periodic-ci-openshift-release-master-ci-4.14-e2e-azure-ovn-upgrade
  • periodic-ci-openshift-release-master-ci-4.14-upgrade-from-stable-4.13-e2e-gcp-ovn-rt-upgrade
  • periodic-ci-openshift-release-master-nightly-4.14-e2e-aws-ovn-serial
  • periodic-ci-openshift-release-master-ci-4.14-e2e-aws-ovn-upgrade
  • periodic-ci-openshift-release-master-nightly-4.14-e2e-metal-ipi-ovn-ipv6
  • periodic-ci-openshift-release-master-nightly-4.14-e2e-metal-ipi-sdn-bm

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/2cbab410-218e-11ee-9a8b-ebdd4b34dd3f-0

@dcbw
Copy link
Contributor Author

dcbw commented Jul 13, 2023 •
edited
Loading

There were some payload failures, at least two of which don't appear to be OVN-related:

periodic-ci-openshift-release-master-ci-4.14-e2e-azure-ovn-upgrade - some upgrade distruption it seems

periodic-ci-openshift-release-master-ci-4.14-upgrade-from-stable-4.13-e2e-gcp-ovn-rt-upgrade - ran into failures with OVNKubernetesResourceRetryFailure because it's an upgrade, which should be fixed by #1747 already

periodic-ci-openshift-release-master-nightly-4.14-e2e-aws-ovn-serial -- etcd and kube-apiservers were upgrading and rejected connections from cluster-network-operator, which then reported an error and didn't retry reconciliation.

@dcbw
Copy link
Contributor Author

dcbw commented Jul 13, 2023

/test all

@dcbw
Copy link
Contributor Author

dcbw commented Jul 13, 2023

/payload 4.14 ci blocking

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jul 13, 2023

@dcbw: trigger 4 job(s) of type blocking for the ci release of OCP 4.14

  • periodic-ci-openshift-release-master-ci-4.14-upgrade-from-stable-4.13-e2e-aws-ovn-upgrade
  • periodic-ci-openshift-release-master-ci-4.14-upgrade-from-stable-4.13-e2e-azure-sdn-upgrade
  • periodic-ci-openshift-release-master-ci-4.14-e2e-gcp-ovn-upgrade
  • periodic-ci-openshift-release-master-ci-4.14-e2e-aws-sdn-serial

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/e64f09c0-2194-11ee-8070-6ec8abd79fce-0

@trozet
Copy link
Contributor

trozet commented Jul 14, 2023

I see:
https://prow.ci.openshift.org/view/gs/origin-ci-test/logs/openshift-ovn-kubernetes-1719-ci-4.14-e2e-gcp-ovn-upgrade/1679519043616247808

{ OVNKubernetesResourceRetryFailure was at or above info for at least 10m58s on platformidentification.JobType{Release:"4.14", FromRelease:"4.14", Platform:"gcp", Architecture:"amd64", Network:"ovn", Topology:"ha"} (maxAllowed=0s): pending for 0s, firing for 10m58s:

W0713 18:07:06.065326       1 obj_retry.go:232] Dropping retry entry for *factory.egressIPPod e2e-replicaset-2344/my-hostname-private-3dbb4afd-3bad-46dc-a380-5d394f154b52-xmdjq: exceeded number of failed attempts
W0713 18:07:06.065518       1 obj_retry.go:232] Dropping retry entry for *factory.egressIPPod e2e-endpointslice-4342/pod-handle-http-request: exceeded number of failed attempts

I0713 18:06:36.065986       1 obj_retry.go:269] Retry object setup: *factory.egressIPPod e2e-replicaset-2344/my-hostname-private-3dbb4afd-3bad-46dc-a380-5d394f154b52-xmdjq
I0713 18:06:36.066176       1 obj_retry.go:302] Removing old object: *factory.egressIPPod e2e-replicaset-2344/my-hostname-private-3dbb4afd-3bad-46dc-a380-5d394f154b52-xmdjq (failed: 14)
I0713 18:06:36.066286       1 obj_retry.go:310] Retry delete failed for *factory.egressIPPod e2e-replicaset-2344/my-hostname-private-3dbb4afd-3bad-46dc-a380-5d394f154b52-xmdjq, will try again later: namespace "e2e-replicaset-2344" not found

looks like delete is failing if it cant find the namespace. we need to fix this. @tssurya @dcbw

@trozet
Copy link
Contributor

trozet commented Jul 14, 2023

oh looks like @kyrtapz already fixed this, it just wasnt in this PR and I guess somehow the aggregator jobs didnt merge this PR into master before running? Anyway I've pulled the OVN bump into #1757 which also has patryk's fix. Lets see how that goes.

@tssurya tssurya mentioned this pull request Jul 14, 2023
Merged
@tssurya
Copy link
Contributor

tssurya commented Jul 14, 2023

yeah pat already fixed that last week..

@dcbw
Copy link
Contributor Author

dcbw commented Jul 17, 2023

Closing since this landed in #1757

@dcbw dcbw closed this Jul 20, 2023
@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Jul 20, 2023

@dcbw: This pull request references Jira Issue OCPBUGS-16120. The bug has been updated to no longer refer to the pull request using the external bug tracker. All external bug links have been closed. The bug has been moved to the NEW state.

Details

In response to this:

New features and improvements:

  • ARP proxy support (needed for CNV)
  • Per-Load Balancer ct_flush
  • Tiered ACLs
  • northd: Update obsolete datapath groups instead of re-creating (perf/scale)

Relevant bug fixes:

  • controller: Handle OpenFlow errors (rhbz#2134880)
  • ovn-controller: fixed port not always set down when unbinding interface (rhbz#2150905)
  • ovn-controller.c: Fix assertion failure during address set update

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@abhat abhat Awaiting requested review from abhat

@trozet trozet Awaiting requested review from trozet

@anuragthehatter anuragthehatter Awaiting requested review from anuragthehatter

Assignees

No one assigned

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. jira/valid-bug Indicates that a referenced Jira bug is valid for the branch this PR is targeting. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@dcbw @openshift-ci-robot @trozet @tssurya @openshift-merge-robot