Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes the "Paused" lifecycle state from the ClusterExtensionRevision API, simplifying the lifecycle model to only support "Active" and "Archived" states. The removal is comprehensive, eliminating the constant definition, related controller logic, and boxcutter integration code.

• Removed the ClusterExtensionRevisionLifecycleStatePaused constant and its documentation
• Removed controller logic that handled paused state (boxcutter WithPaused option)
• Simplified revision filtering logic to only skip archived revisions

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
api/v1/clusterextensionrevision_types.go	Removed the ClusterExtensionRevisionLifecycleStatePaused constant and its documentation
internal/operator-controller/controllers/clusterextensionrevision_controller.go	Removed the conditional logic that added boxcutter.WithPaused option for paused revisions
internal/operator-controller/controllers/boxcutter_reconcile_steps.go	Simplified from a switch statement to a simple conditional check, now only skipping archived revisions

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.