openshift / origin

Conformance test suite for OpenShift
http://www.openshift.org
Apache License 2.0
8.49k stars 4.7k forks source link

SDN-4930: ovn-k, udn crd e2e: Fix status report consumer assertion #29287

Closed ormergi closed 1 week ago

ormergi commented 1 week ago

Following the incoming changes of the CUDN controller by origin/ovn-kubernetes#2345, the assertion on the UDN status is no longer valid.

This PR fix the assertion with the expected condition message.

Note to reviewer: Merging this PR means there might be intermediate time where techpreview lane will fail due to missing bits of https://github.com/openshift/ovn-kubernetes/pull/2345 Or if https://github.com/openshift/ovn-kubernetes/pull/2345 merged without this PR changes.

PR converted to draft state to avoid burning CI time

ormergi commented 1 week ago

/cc @maiqueb

openshift-ci[bot] commented 1 week ago

Skipping CI for Draft Pull Request. If you want CI signal for your change, please convert it to an actual PR. You can still manually trigger a test run with /test all

openshift-ci[bot] commented 1 week ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: maiqueb, ormergi Once this PR has been reviewed and has the lgtm label, please assign adambkaplan for approval. For more information see the Kubernetes Code Review Process.

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

Needs approval from an approver in each of these files: - **[test/extended/networking/OWNERS](https://github.com/openshift/origin/blob/master/test/extended/networking/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
tssurya commented 1 week ago

/retitle SDN-4930: ovn-k, udn crd e2e: Fix status report consumer assertion