Restore tekton task memory limits#3274
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughMemory requests and limits in a Tekton Task stepTemplate were increased from 2Gi to 8Gi for both requests and limits. ChangesResource Configuration Update
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Some tenants are experiencing OOM issues due to the much lower limit, so let's restore it to the original value.
b3b312c to
fedc7f9
Compare
|
As discussed on slack, the two teams that need this (openstack and ocp-art) have already modified their pipeline defaults to increase the memory resources. Not sure if any other teams will be impacted, so we might decide this change is actually not needed now. |
Codecov Report✅ All modified and coverable lines are covered by tests.
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
|
Set it to draft, since I think the current plan is to abandon. |
Some tenants are experiencing OOM issues due to the much lower limit, so let's restore it to the original value.