Skip to content

[OSPRH-18744] Handle variant apiVersions for refData cleanup#1554

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
abays:fix-OSPRH-18744
Aug 11, 2025
Merged

[OSPRH-18744] Handle variant apiVersions for refData cleanup#1554
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
abays:fix-OSPRH-18744

Conversation

@abays

@abays abays commented Aug 6, 2025

Copy link
Copy Markdown
Contributor

Some refData in Operator resources can lack a group in its apiVersion. We need to adjust our Operator clean-up logic to account for this.

Jira: https://issues.redhat.com/browse/OSPRH-18744

@abays abays requested a review from stuggi August 6, 2025 10:54
@openshift-ci openshift-ci Bot requested review from fultonj and slagle August 6, 2025 10:54
@openshift-ci openshift-ci Bot added the approved label Aug 6, 2025
@abays

abays commented Aug 8, 2025

Copy link
Copy Markdown
Contributor Author

@abays: The following test 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/openstack-operator-build-deploy-kuttl 83adc21 link true /test openstack-operator-build-deploy-kuttl

Full PR test history. Your PR dashboard.

{  failed to wait for the created cluster claim to become ready: timed out waiting for the condition}

/test openstack-operator-build-deploy-kuttl

@stuggi stuggi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci

openshift-ci Bot commented Aug 11, 2025

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: abays, stuggi

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

@stuggi

stuggi commented Aug 11, 2025

Copy link
Copy Markdown
Contributor

/cherry-pick 18.0-fr3

@openshift-cherrypick-robot

Copy link
Copy Markdown

@stuggi: once the present PR merges, I will cherry-pick it on top of 18.0-fr3 in a new PR and assign it to you.

Details

In response to this:

/cherry-pick 18.0-fr3

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.

@openshift-merge-bot openshift-merge-bot Bot merged commit ebf902f into openstack-k8s-operators:main Aug 11, 2025
9 checks passed
@openshift-cherrypick-robot

Copy link
Copy Markdown

@stuggi: new pull request created: #1557

Details

In response to this:

/cherry-pick 18.0-fr3

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.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants