Skip to content

ROX-33270: Only set spec.replicas if autoscaling is disabled.#19199

Merged
porridge merged 1 commit intomasterfrom
mc/status-controller-deployment-hpa-fix-minimal
Feb 26, 2026
Merged

ROX-33270: Only set spec.replicas if autoscaling is disabled.#19199
porridge merged 1 commit intomasterfrom
mc/status-controller-deployment-hpa-fix-minimal

Conversation

@porridge
Copy link
Contributor

@porridge porridge commented Feb 25, 2026

Description

A change to our Helm templates which prevents our operator from playing ping-pong with the horizontal pod autoscaler (HPA). Don't set spec.replicas for deployments in case auto-scaling is enabled.

This is simply one of the commits in #19194

User-facing documentation

Testing and quality

  • the change is production ready: the change is GA, or otherwise the functionality is gated by a feature flag
  • CI results are inspected

Automated testing

  • modified existing tests

How I validated my change

Rolled out into a dev cluster and observed.

@porridge porridge requested a review from mclasmeier February 25, 2026 18:27
@porridge porridge requested a review from a team as a code owner February 25, 2026 18:27
@rhacs-bot
Copy link
Contributor

Images are ready for the commit at 1efff08.

To use with deploy scripts, first export MAIN_IMAGE_TAG=4.11.x-181-g1efff08ecd.

@porridge porridge added the auto-retest PRs with this label will be automatically retested if prow checks fails label Feb 25, 2026
@codecov
Copy link

codecov bot commented Feb 25, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 49.55%. Comparing base (02aadd3) to head (1efff08).
⚠️ Report is 6 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #19199      +/-   ##
==========================================
- Coverage   49.56%   49.55%   -0.01%     
==========================================
  Files        2675     2675              
  Lines      201820   201820              
==========================================
- Hits       100023   100008      -15     
- Misses      94340    94352      +12     
- Partials     7457     7460       +3     
Flag Coverage Δ
go-unit-tests 49.55% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@rhacs-bot
Copy link
Contributor

/retest

@rhacs-bot
Copy link
Contributor

/retest

@porridge porridge merged commit bf4ac42 into master Feb 26, 2026
110 checks passed
@porridge porridge deleted the mc/status-controller-deployment-hpa-fix-minimal branch February 26, 2026 08:12
rhacs-bot pushed a commit that referenced this pull request Feb 26, 2026
Co-authored-by: Moritz Clasmeier <mclasmeier@redhat.com>
(cherry picked from commit bf4ac42)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/helm auto-retest PRs with this label will be automatically retested if prow checks fails backport release-4.10

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants