Skip to content

fix: raise manager memory limit to 2Gi (OOMKilled crash-loop)#138

Merged
1995parham merged 1 commit into
mainfrom
fix/manager-memory-limit
Jun 20, 2026
Merged

fix: raise manager memory limit to 2Gi (OOMKilled crash-loop)#138
1995parham merged 1 commit into
mainfrom
fix/manager-memory-limit

Conversation

@1995parham

Copy link
Copy Markdown
Member

Problem

A fresh manager pod OOMKilled crash-loops (43 restarts observed): 512Mi can't hold the cold-start informer cache spike — the controllers watch Secrets/ConfigMaps/Namespaces cluster-wide, and on a large (225-node) cluster the initial LIST exceeds the limit. The long-running pod only survived because its cache was warmed when the cluster was smaller; any restart now crash-loops.

Manager lastState.terminated: reason=OOMKilled exitCode=137, crash begins right after cache/leader-election startup, before any reconcile.

Fix

Raise the manager limit 512Mi → 2Gi and request 128Mi → 512Mi. Config-only; synced by the spcld-argocd-complementary-operator Argo app.

(Follow-up worth considering separately: scope the Secret/ConfigMap watches with field/namespace selectors to cut steady-state memory.)

🤖 Generated with Claude Code

The manager OOMKilled-crash-looped on a fresh pod: 512Mi can't hold the
cold-start informer cache spike (cluster-wide Secret/ConfigMap/Namespace
caches) on large clusters. The previous pod only survived because its
cache was warmed long ago. Raise limit 512Mi->2Gi, request 128Mi->512Mi.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@codecov

codecov Bot commented Jun 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 49.16%. Comparing base (f539bb5) to head (f84e807).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #138      +/-   ##
==========================================
+ Coverage   48.22%   49.16%   +0.93%     
==========================================
  Files           8        8              
  Lines         960      960              
==========================================
+ Hits          463      472       +9     
+ Misses        433      427       -6     
+ Partials       64       61       -3     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@1995parham 1995parham merged commit 452ba85 into main Jun 20, 2026
5 checks passed
@1995parham 1995parham deleted the fix/manager-memory-limit branch June 20, 2026 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant