Skip to content

Conversation

@stephenfin
Copy link
Contributor

Pull in the most recent version which includes the v1beta2 API required by CAPO v0.13.x (and likely others in the future).

This can be treated as a pre-requisite for #10090.

Pull in the most recent version which includes the v1beta2 API required
by CAPO v0.13.x (and likely others in the future).

Signed-off-by: Stephen Finucane <[email protected]>
@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Nov 19, 2025
@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Nov 19, 2025

@stephenfin: This pull request references OSASINFRA-3965 which is a valid jira issue.

In response to this:

Pull in the most recent version which includes the v1beta2 API required by CAPO v0.13.x (and likely others in the future).

This can be treated as a pre-requisite for #10090.

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 openshift-eng/jira-lifecycle-plugin repository.

@stephenfin
Copy link
Contributor Author

/retest-required

@stephenfin
Copy link
Contributor Author

/retest

@patrickdillon
Copy link
Contributor

/test e2e-nutanix-ovn e2e-openstack-ovn e2e-powervs-ovn

@patrickdillon
Copy link
Contributor

/test e2e-ibmcloud-ovn

@patrickdillon
Copy link
Contributor

/test e2e-powervs-capi-ovn

@patrickdillon
Copy link
Contributor

/approve
/lgtm

Let's wait to make sure we don't break any of these platforms before adding verified

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Nov 20, 2025
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 20, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: patrickdillon

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 /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 Nov 20, 2025
@stephenfin
Copy link
Contributor Author

e2e-azure-ovn-shared-vpc

This job appears to be permafailing currently.

e2e-nutanix-ovn

Also permafailing currently.

e2e-openstack-ovn

This is still running at time of comment, but sadly it's not in great shape either (we are working on it!)

e2e-ibmcloud-ovn

Also still running. This could be 50:50 (well, closer to 25:75). TBC.

e2e-powervs-capi-ovn

This job doesn't ever appear to have passed.

e2e-gcp-custom-dns

This should have passed and only two tests failed. I'll recheck on the assumption that they are flakes.

/test e2e-gcp-custom-dns

Overall, I think this should be good to go, but I'll hold of adding verified until you've seen this and agreed/disagreed, @patrickdillon

@patrickdillon
Copy link
Contributor

/verified by multiple e2e-tests

I reviewed all the marginal test cases to ensure there are no regressions:
nutanix: infrastructure provisioned successfully
ibmcloud: install completed, e2es still in progress
powervs: seems the test is broken but ibmcloud uses the same provider
openstack: well, I defer to you!

Thanks for checking these so carefully

@openshift-ci-robot openshift-ci-robot added the verified Signifies that the PR passed pre-merge verification criteria label Nov 20, 2025
@openshift-ci-robot
Copy link
Contributor

@patrickdillon: This PR has been marked as verified by multiple e2e-tests.

In response to this:

/verified by multiple e2e-tests

I reviewed all the marginal test cases to ensure there are no regressions:
nutanix: infrastructure provisioned successfully
ibmcloud: install completed, e2es still in progress
powervs: seems the test is broken but ibmcloud uses the same provider
openstack: well, I defer to you!

Thanks for checking these so carefully

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 openshift-eng/jira-lifecycle-plugin repository.

@patrickdillon patrickdillon merged commit 993a7a5 into openshift:main Nov 20, 2025
26 of 34 checks passed
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Nov 20, 2025

@stephenfin: 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-powervs-capi-ovn f199ccb link false /test e2e-powervs-capi-ovn
ci/prow/e2e-nutanix-ovn f199ccb link false /test e2e-nutanix-ovn
ci/prow/e2e-azure-ovn-shared-vpc f199ccb link false /test e2e-azure-ovn-shared-vpc
ci/prow/e2e-azurestack f199ccb link false /test e2e-azurestack

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-sigs/prow repository. I understand the commands that are listed here.

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

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. lgtm Indicates that a PR is ready to be merged. verified Signifies that the PR passed pre-merge verification criteria

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants