Prevent downgrade of thanos receive PVC to 1Gi in jenkins e2e#2558
Prevent downgrade of thanos receive PVC to 1Gi in jenkins e2e#2558dbuchanaRH wants to merge 1 commit into
Conversation
Signed-off-by: Daniel Buchanan <dbuchana@redhat.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (4)
📝 WalkthroughWalkthroughThis change updates the ChangesExample config storage size updates
Estimated code review effort: 1 (Trivial) | ~2 minutes Storage sizing bumped fourfold across the fleet of example manifests — like swapping a compact spare tire for a proper one on all four corners at once. Clean, consistent, and no logic to untangle here. Ship it! ✅🔍 🚥 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 |
|
@dbuchanaRH: The following test 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. |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dbuchanaRH, thibaultmg The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
e2e tests are deployed with thanos receive storage size at 10Gi. In jenkins QE regressions the test RHACM4K-43019 can run which downgrades the PVC size for thanos receive to 1Gi. If the test suite was running for an extended period of time this will push the thanos receive pods into an error state.
As mentioned in #2257 increasing the PV storage will delay the problem for now. There may come a need for another solution.
After these changes thanos receive storage size should stay at 10Gi through the duration of the e2e suite. receiveStorageSize is not changed for unit tests or for kind e2e tests i.e.
Summary by CodeRabbit
1Gito10Giin example configurations.