⚠️ (Boxcutter Runtime) Revision discovery now uses label-based cache lookups, removing Spec.Previous and preventing orphaned revisions on delete. by camilamacedo86 · Pull Request #2315 · operator-framework/operator-controller

AI review requested due to automatic review settings

November 9, 2025 00:17

@camilamacedo86 camilamacedo86 changed the title ✨ (chore): (Boxcutter): Use label selector for listing previous revisions WIP ✨ (chore): (Boxcutter): Use label selector for listing previous revisions

Nov 9, 2025

@camilamacedo86 camilamacedo86 changed the title WIP ✨ (chore): (Boxcutter): Use label selector for listing previous revisions WIP ✨ (chore): (Boxcutter): se label-based cache for revision lookups instead of explicit chains;

Nov 9, 2025

@camilamacedo86 camilamacedo86 changed the title WIP ✨ (chore): (Boxcutter): se label-based cache for revision lookups instead of explicit chains; ✨ (Valid for Boxcutter Runtime Only) switched to label-based revision lookups — faster, safer, and no orphans left behind.

Nov 9, 2025

@camilamacedo86 camilamacedo86 changed the title ✨ (Valid for Boxcutter Runtime Only) switched to label-based revision lookups — faster, safer, and no orphans left behind. WIP ✨ (Valid for Boxcutter Runtime Only) switched to label-based revision lookups — faster, safer, and no orphans left behind.

Nov 9, 2025

Copilot AI review requested due to automatic review settings

November 9, 2025 13:10

Copilot AI review requested due to automatic review settings

November 9, 2025 18:24

camilamacedo86

camilamacedo86

Copilot AI review requested due to automatic review settings

November 17, 2025 17:38

Copilot AI review requested due to automatic review settings

November 17, 2025 18:13

@camilamacedo86

…it chains

Instead of tracking revision history through Spec.Previous fields, we now
find related revisions using labels and the controller-runtime cache. This
is more efficient and works better with controller-runtime's caching layer.

To support this change, the Helm-to-Boxcutter migration now sets
ownerReferences on migrated revisions, ensuring they work identically
to newly created revisions.

Assisted-by: Cursor

pedjak