fix(tests): increase memory limits for race-detector builds#19951
fix(tests): increase memory limits for race-detector builds#19951
Conversation
The busybox-style binary consolidation (ROX-33958) runs init() for all components at startup. Under the race detector's ~5-10x memory multiplier this causes OOMKills for admission-control (500Mi limit) and config-controller (128Mi limit). Override memory limits post-deploy when IS_RACE_BUILD is set, following the same pattern used for OpenShift CPU overrides (ROX-5334). - config-controller: 128Mi → 512Mi (race builds only) - admission-control: 500Mi → 2Gi (race builds only) Default Helm chart values are unchanged. Generated with assistance from AI Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
Skipping CI for Draft Pull Request. |
🚀 Build Images ReadyImages are ready for commit 1ee3f01. To use with deploy scripts: export MAIN_IMAGE_TAG=4.11.x-630-g1ee3f01197 |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #19951 +/- ##
=======================================
Coverage 49.56% 49.56%
=======================================
Files 2764 2764
Lines 208357 208357
=======================================
Hits 103269 103269
Misses 97436 97436
Partials 7652 7652
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
…yment The config-controller deployment names its container 'manager', not 'config-controller'. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
… resources The operator reconciles and reverts kubectl resource overrides. For operator-managed deployments (OCP jobs), patch the Central/SecuredCluster CRs directly. For helm/kubectl deployments (GKE jobs), keep using kubectl set resources. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
/test gke-qa-e2e-tests |
davdhacs
added
the
auto-retest
|
/retest-required |
|
/retest |
3 similar comments
|
/retest |
|
/retest |
|
/retest |
davdhacs
removed
the
auto-retest
|
@davdhacs: The following tests failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
davdhacs
force-pushed
the
davdhacs/fix-rcd-oom-race-overrides
branch
from
9328fc6 to
1ee3f01
Compare
2 participants
Description
ROX-33958 busybox binary consolidation causes OOMKills for admission-control and config-controller in race-detector nightly builds. Increases memory limits post-deploy when
IS_RACE_BUILDis set.Testing and quality
Automated testing
How I validated my change
Same pattern as existing OpenShift CPU override on line 424.
🤖 Generated with Claude Code