fix(rabbitmq): raise default resources preset to u1.nano - #3936
fix(rabbitmq): raise default resources preset to u1.nano#3936gettyeuro (yankawai) wants to merge 1 commit into
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (7)
Included review availability: Your plan provides up to 8 included reviews per hour; 4 remain after this review. 📝 WalkthroughWalkthroughRabbitMQ now uses ChangesRabbitMQ resource configuration
Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to The PR raises RabbitMQ's default resource preset while preserving the legacy explicit override and covering the supported configurations with tests. No actionable merge-blocking risk remains beyond normal checks and review. Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Signed-off-by: Yan Bondarenko <202671653+yankawai@users.noreply.github.com>
cc95aa3 to
d5ff93b
Compare
What this PR does
Raises the RabbitMQ default resource preset from
t1.nano(128Mi memory) tou1.nano(1Gi memory). An explicitt1.nanoremains valid, and existing application CRs that already persistresourcesPreset: t1.nanoare not resized automatically.The original default-sized RabbitMQ 4.2 pod on Cozystack 1.6.1 was OOMKilled with exit 137, reached 10 restarts, and never became Ready. A temporary
u1.nanocanary reached 3/3 Ready with zero restarts; its working-set snapshots were 114–121Mi. The canary and its storage were removed after validation.The values, generated schema, ApplicationDefinition, typed API, and README use the same default. Tests cover the new default, an explicit legacy override, and explicit resources.
Validation:
helm unittest packages/apps/rabbitmq: 3/3 passed.make -C packages/apps/rabbitmq generate: no diff.Screenshots
Not a UI change.
Downstream repositories
The default is restated by the Terraform provider, so the linked provider PR updates it and documents the state-plan implication. No other trigger-map entry matches this diff.
Release note