🌱 ClusterExtensionRevision conditions

[perdasilva]

Description

Reviewer Checklist

• API Go Documentation
• Tests: Unit Tests (and E2E Tests, if appropriate)
• Comprehensive Commit Messages
• Links to related GitHub Issue(s)

[netlify]

✅ Deploy Preview for olmv1 ready!

Name	Link
🔨 Latest commit	295c589
🔍 Latest deploy log	https://app.netlify.com/projects/olmv1/deploys/69143dd0945be900085cc7c0
😎 Deploy Preview	https://deploy-preview-2281--olmv1.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[perdasilva]

/hold wip

[codecov]

Codecov Report

❌ Patch coverage is 84.73520% with 49 lines in your changes missing coverage. Please review.
✅ Project coverage is 74.41%. Comparing base (d204888) to head (295c589).

Files with missing lines	Patch %	Lines
...controllers/clusterextensionrevision_controller.go	75.00%	20 Missing and 3 partials ⚠️
api/v1/zz_generated.deepcopy.go	38.88%	10 Missing and 1 partial ⚠️
...er/controllers/clusterextension_reconcile_steps.go	89.00%	8 Missing and 3 partials ⚠️
...ontroller/controllers/boxcutter_reconcile_steps.go	95.23%	2 Missing and 2 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2281      +/-   ##
==========================================
- Coverage   74.42%   74.41%   -0.01%     
==========================================
  Files          91       93       +2     
  Lines        7057     7184     +127     
==========================================
+ Hits         5252     5346      +94     
- Misses       1393     1417      +24     
- Partials      412      421       +9     

Flag	Coverage Δ
e2e	45.55% <30.21%> (-0.31%)	⬇️
experimental-e2e	48.66% <68.53%> (+0.20%)	⬆️
unit	58.12% <50.46%> (-0.39%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[openshift-ci]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign perdasilva for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

• OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[Copilot]

Pull Request Overview

This PR refactors the ClusterExtension reconciliation process to use a step-based architecture and updates condition handling to use Available and Progressing conditions instead of the legacy Installed condition. The changes support better status reporting from ClusterExtensionRevision objects and introduce an ActiveRevisions status field.

Key changes:

• Introduces a step-based reconciliation pattern using ReconcileStepFunc to break down the reconcile logic into composable steps
• Updates CRD printer columns and status types to use Available instead of Installed condition
• Adds Progressing condition to ClusterExtensionRevision with new reasons like RollingOut, RolloutError, and RolledOut
• Introduces ActiveRevisions status field to track active revisions in ClusterExtension status
• Adds new test data for v1.0.2 test-operator bundle with httpd-based container setup

Reviewed Changes

Copilot reviewed 30 out of 30 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
internal/operator-controller/controllers/clusterextension_reconcile_steps.go	New file implementing generic reconcile steps for finalizers, revision states, metadata retrieval, bundle unpacking, and application
internal/operator-controller/controllers/boxcutter_reconcile_steps.go	New file with boxcutter-specific reconcile steps including storage migration and condition mirroring
internal/operator-controller/controllers/clusterextension_controller.go	Refactored to remove inline reconcile logic, moved to step-based architecture with ReconcileSteps
internal/operator-controller/controllers/clusterextensionrevision_controller.go	Updated condition handling to use new helper methods and improved status messages with version information
api/v1/clusterextensionrevision_types.go	Added helper methods for marking revision status and new condition constants
api/v1/clusterextension_types.go	Added RevisionStatus type and ActiveRevisions field to status
CRD manifests	Updated printer columns from Installed to Available and added ActiveRevisions schema
Test files	Updated to use new reconciler setup pattern with options, updated expected conditions
Test data	Added v1.0.2 bundle with httpd container setup and updated v1.0.0/v1.2.0 bundles

---

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

[Copilot]

Pull Request Overview

Copilot reviewed 26 out of 27 changed files in this pull request and generated 2 comments.

---

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

[Copilot]

Pull Request Overview

Copilot reviewed 28 out of 29 changed files in this pull request and generated 5 comments.

---

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

[Copilot]

Pull Request Overview

Copilot reviewed 28 out of 29 changed files in this pull request and generated no new comments.

---

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