Skip to content

[18.0-fr3] Add interface to allow customize of operator deployments#1526

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:18.0-fr3from
openshift-cherrypick-robot:cherry-pick-1519-to-18.0-fr3
Jul 9, 2025
Merged

[18.0-fr3] Add interface to allow customize of operator deployments#1526
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:18.0-fr3from
openshift-cherrypick-robot:cherry-pick-1519-to-18.0-fr3

Conversation

@openshift-cherrypick-robot

@openshift-cherrypick-robot openshift-cherrypick-robot commented Jul 9, 2025

Copy link
Copy Markdown

This is an automated cherry-pick of #1519

/assign stuggi

Jira: OSPRH-17785

Adds an interface which allows to customize the following
parameters of a controller-manager operator deployment
- change the replica (0/1)
- change the resources (requests/limits) of the manager container

It keeps the current default resource requests as is. For easier
gotemplate it this adds an internal custom struct type, which allows
to just loop over a list of operator.Operator for service operators
and a dedicated instance for the openstack-operator-controller-manager
and the rabbitmq-cluster-operator-manager.

E.g. changing the default cpu limits for the infra operator and
disabling the openstack-baremetal operator would be:
~~~
...
spec:
  operatorOverrides:
  - name: infra
    controllerManager:
      resources:
        limits:
          cpu: 600m
          memory: 2Gi
    replicas: 1
  - name: openstack-baremetal
    replicas: 0
~~~

To add a new service operator has the following high level steps:
- in the openstack_types.go api
  - add a new const with the name
  - add an entry for the defaults in the OperatorList var
  - add the name also to the Enum of the OperatorSpec.Name
- (optional) if any specific env var need be passed to the operator,
  e.g. related images list, or to enable webhooks, check the
  applyOperator() in the openstack_controller.go .

Jira: OSPRH-17785

Signed-off-by: Martin Schuppert <mschuppert@redhat.com>
@stuggi

stuggi commented Jul 9, 2025

Copy link
Copy Markdown
Contributor

/lgmt

@openshift-ci

openshift-ci Bot commented Jul 9, 2025

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

Approval requirements bypassed by manually added approval.

This pull-request has been approved by: openshift-cherrypick-robot

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

@openshift-merge-bot openshift-merge-bot Bot merged commit 76c45a9 into openstack-k8s-operators:18.0-fr3 Jul 9, 2025
7 checks passed
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.

2 participants