Skip to content

Revert "Bump operator-lint"#333

Closed
dprince wants to merge 1 commit into
openstack-k8s-operators:mainfrom
dprince:revert_lint
Closed

Revert "Bump operator-lint"#333
dprince wants to merge 1 commit into
openstack-k8s-operators:mainfrom
dprince:revert_lint

Conversation

@dprince

@dprince dprince commented Sep 30, 2025

Copy link
Copy Markdown
Contributor

This reverts commit 4fdb30a.

This seems to be causing functional test failures in openstack-operator:

OpenStackDataPlaneNodeSet.dataplane.openstack.org "ab1ba43d-50e8-463c-9764-3" is invalid: spec.baremetalSetTemplate.automatedCleaningMode: Unsupported value: "": supported values: "metadata", "disabled"

This reverts commit 4fdb30a.

This seems to be causing functional test failures in openstack-operator:

OpenStackDataPlaneNodeSet.dataplane.openstack.org "ab1ba43d-50e8-463c-9764-3" is invalid: spec.baremetalSetTemplate.automatedCleaningMode: Unsupported value: "": supported values: "metadata", "disabled"
@openshift-ci openshift-ci Bot requested review from frenzyfriday and rabi September 30, 2025 19:05
@openshift-ci

openshift-ci Bot commented Sep 30, 2025

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dprince

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

@softwarefactory-project-zuul

Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/fb8eef856320490b81e0fd1825504c97

openstack-baremetal-operator-content-provider FAILURE in 19m 20s
⚠️ openstack-baremetal-operator-crc-podified-edpm-baremetal SKIPPED Skipped due to failed job openstack-baremetal-operator-content-provider

@rabi

rabi commented Sep 30, 2025

Copy link
Copy Markdown
Contributor

We should have merged openstack-k8s-operators/openstack-operator#1622 before #328. Once 1622 merges we would not have the issue.

@dprince

dprince commented Sep 30, 2025

Copy link
Copy Markdown
Contributor Author

Ideally #1622 fixes this. Leaving in case there is an issue as we need to get things synced soon

@dprince

dprince commented Sep 30, 2025

Copy link
Copy Markdown
Contributor Author

@rabi ack. we figure this out late today and I had not seen your other PR yet.

@rabi

rabi commented Oct 1, 2025

Copy link
Copy Markdown
Contributor

openstack-operator functional tests have been fixed with openstack-k8s-operators/openstack-operator#1622. No need to revert this.

@rabi rabi closed this Oct 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants