CLOUD-727 fix OOMKill in demand-backup-if-unhealthy#2309
Conversation
There was a problem hiding this comment.
Pull request overview
This PR aims to prevent OOMKills in the demand-backup-if-unhealthy E2E scenario by adjusting the PerconaServerMongoDB CR used by that test.
Changes:
- Removed CPU/memory
resources.limitsfrom thedemand-backup-if-unhealthyE2E CR config. - Kept the existing CPU/memory
resources.requestsunchanged.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| memory: 300Mi | ||
| requests: | ||
| cpu: 100m | ||
| memory: 100Mi |
There was a problem hiding this comment.
This test CR now sets only resources.requests and removes resources.limits. Most other e2e CR configs in this repo set both requests and limits (e.g., e2e-tests/conf/some-name-rs0.yml and e2e-tests/demand-backup-incremental-aws/conf/some-name.yml). Dropping limits can make the e2e test less portable (namespaces with a LimitRange that requires limits will reject the workload) and can allow unbounded memory usage on CI nodes. Consider keeping limits but raising the memory limit to avoid OOMKills (and potentially adjusting requests to a more realistic baseline).
| memory: 100Mi | |
| memory: 100Mi | |
| limits: | |
| cpu: 300m | |
| memory: 1Gi |
commit: 4eef1de |
CHANGE DESCRIPTION
Problem:
Short explanation of the problem.
Cause:
Short explanation of the root cause of the issue if applicable.
Solution:
Short explanation of the solution we are providing with this PR.
CHECKLIST
Jira
Needs Doc) and QA (Needs QA)?Tests
compare/*-oc.yml)?Config/Logging/Testability