:bug: Fix Boxcutter manifest ordering inconsistency by tmshort · Pull Request #2329 · operator-framework/operator-controller

perdasilva

@tmshort tmshort changed the title 🐛 Fix duplicate ClusterExtensionRevisions during Helm-to-Boxcutter migr… 🐛 Fix Boxcutter manifest ordering inconsistency

Nov 17, 2025

camilamacedo86

perdasilva

perdasilva

@openshift-ci openshift-ci bot added the approved

Indicates a PR has been approved by an approver from all required OWNERS files.

label

Nov 18, 2025

perdasilva

@tmshort @claude

Issue:
Manifest ordering inconsistency: CRDs from Helm release manifest and
bundle manifest appeared in different orders, causing PhaseSort to
produce different phase structures even though they contained the same
objects.

Solution:
Added deterministic sorting in PhaseSort (phase.go):
 - Sort objects within each phase by Group, Version, Kind, Namespace, Name
 - Ensures consistent phase structure regardless of input order
 - Critical for comparing revisions from different sources

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Todd Short <tshort@redhat.com>