Skip to content

Use OpenStack CR operatorOverrides to scale down placement operator#411

Open
karelyatin wants to merge 1 commit into
openstack-k8s-operators:mainfrom
karelyatin:operator_override
Open

Use OpenStack CR operatorOverrides to scale down placement operator#411
karelyatin wants to merge 1 commit into
openstack-k8s-operators:mainfrom
karelyatin:operator_override

Conversation

@karelyatin
Copy link
Copy Markdown
Contributor

Switch from CSV-based scaling to using OpenStack CR operatorOverrides
feature for scaling down the placement operator. This avoids the need
to disable the openstack init operator and provides better integration
with the OpenStack operator management system.

Switch from CSV-based scaling to using OpenStack CR operatorOverrides
feature for scaling down the placement operator. This avoids the need
to disable the openstack init operator and provides better integration
with the OpenStack operator management system.

Signed-off-by: Yatin Karel <ykarel@redhat.com>
@openshift-ci openshift-ci Bot requested review from gibizer and stuggi April 21, 2026 15:07
@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Apr 21, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

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

The full list of commands accepted by this bot can be found 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

@fmount
Copy link
Copy Markdown
Contributor

fmount commented Apr 30, 2026

/lgtm

@SeanMooney
Copy link
Copy Markdown
Collaborator

oh i do not have an env to test this myself currently but this makes sense
i was going to say we are not allwo to have a depency liek this but this is actully just a chagne toe the run_with_webhook integration for local deployment not the operator its self so that is completely fine in the make/bash script we use here.

it would have nice to call that out a little more explcity in the PR body

im not going to appove this mearly be cause we do not have ci testign fo this as far as im aware and i have not verified it works locally but i have no objection to this change.

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.

3 participants