🐛 OCPBUGS-77972: fix: aggregate phase errors to handle SSA migration conflicts#2639
Conversation
✅ Deploy Preview for olmv1 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Pull request overview
Enables Boxcutter’s phase error aggregation during ClusterObjectSet reconciliation so SSA migration conflicts across phases can be reported together instead of failing fast on the first phase.
Changes:
- Add
boxcutter.WithAggregatePhaseReconcileErrors()to the revision reconcile options constructed inbuildBoxcutterPhases.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2639 +/- ##
==========================================
- Coverage 68.97% 68.95% -0.02%
==========================================
Files 139 139
Lines 9891 9892 +1
==========================================
- Hits 6822 6821 -1
- Misses 2561 2562 +1
- Partials 508 509 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
/approve |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: rashmigottipati, tmshort 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 |
a34d269
into
operator-framework:main
No description provided.