Skip to content

[Feat] Operator: HPA config enabled#366

Merged
Pavan-SAP merged 1 commit intomainfrom
hpa
Mar 29, 2026
Merged

[Feat] Operator: HPA config enabled#366
Pavan-SAP merged 1 commit intomainfrom
hpa

Conversation

@Pavan-SAP
Copy link
Copy Markdown
Member

This change enables explicit support for HPA.
Doing this within the operator allows us to also cleanup HPAs for older versions (as this is owned by the version).

@Pavan-SAP Pavan-SAP added the build-pr-images Set this label on PRs to build docker images for changes label Mar 23, 2026
@Pavan-SAP Pavan-SAP added build-pr-images Set this label on PRs to build docker images for changes and removed build-pr-images Set this label on PRs to build docker images for changes labels Mar 27, 2026
@Pavan-SAP Pavan-SAP force-pushed the hpa branch 2 times, most recently from b08df77 to a476c93 Compare March 27, 2026 20:00
@Pavan-SAP Pavan-SAP added build-pr-images Set this label on PRs to build docker images for changes and removed build-pr-images Set this label on PRs to build docker images for changes labels Mar 27, 2026
This change enables explicit support for HPA.
Doing this within the operator allows us to also cleanup HPAs for older
versions (as this is owned by the version).

We had to create our own HPA spec as scaleTargerRef is required and the
one from the k8s client does not allow to omit it.
We cannnot let users specify it as this needs to be set by the operator
to the correct deployment values.
@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

@anirudhprasad-sap anirudhprasad-sap left a comment

Choose a reason for hiding this comment

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

👍

@Pavan-SAP Pavan-SAP merged commit afbcd12 into main Mar 29, 2026
9 checks passed
@Pavan-SAP Pavan-SAP deleted the hpa branch March 29, 2026 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build-pr-images Set this label on PRs to build docker images for changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants