🌱 Boxcutter Library Update

[dtfranz]

Description

Updates boxcutter library to (breaking) version v0.11.0.

Reviewer Checklist

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

[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 joelanford for approval. For more information see the Code Review Process.

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

Details

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

Updates the experimental Boxcutter-based reconciliation path to align with a newer Boxcutter library release and accompanying dependency bumps.

Changes:

• Bump pkg.package-operator.run/boxcutter to v0.11.0 and adjust the ClusterExtensionRevision controller to use the updated revision/phase building APIs.
• Update several Kubernetes module versions in go.mod/go.sum.
• Add a top-level @WIP tag to the BoxcutterRuntime e2e feature.

Reviewed changes

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

Show a summary per file

File	Description
test/e2e/features/revision.feature	Adds an @WIP tag to the ClusterExtensionRevision e2e feature file.
internal/operator-controller/controllers/revision_engine_factory.go	Updates Boxcutter engine construction to match new comparator/object-engine wiring.
internal/operator-controller/controllers/clusterextensionrevision_controller.go	Refactors revision construction to use NewRevisionWithOwner and phases-based helpers.
go.mod	Bumps Boxcutter and several k8s.io/* module versions.
go.sum	Updates module checksums for the Boxcutter bump.

---

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

[netlify]

✅ Deploy Preview for olmv1 ready!

Name	Link
🔨 Latest commit	e1a07cf
🔍 Latest deploy log	https://app.netlify.com/projects/olmv1/deploys/69b20cff46c6c80008ede526
😎 Deploy Preview	https://deploy-preview-2554--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.

[Copilot]

Pull request overview

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

Comments suppressed due to low confidence (1)

internal/operator-controller/controllers/clusterextensionrevision_controller.go:149

• The error message here still says "converting to boxcutter revision", but the helper was renamed to buildBoxcutterPhases and now returns phases/options (not a revision). Updating the message will make failures easier to understand and debug (e.g., "building boxcutter phases").

	phases, opts, err := c.buildBoxcutterPhases(ctx, cer)
	if err != nil {
		setRetryingConditions(cer, err.Error())
		return ctrl.Result{}, fmt.Errorf("converting to boxcutter revision: %v", err)
	}

---

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

[codecov]

Codecov Report

❌ Patch coverage is 95.45455% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 64.30%. Comparing base (6f23a79) to head (e1a07cf).
⚠️ Report is 5 commits behind head on main.

Files with missing lines	Patch %	Lines
...-controller/controllers/revision_engine_factory.go	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2554      +/-   ##
==========================================
- Coverage   68.60%   64.30%   -4.30%     
==========================================
  Files         131      131              
  Lines        9330     9333       +3     
==========================================
- Hits         6401     6002     -399     
- Misses       2438     2854     +416     
+ Partials      491      477      -14     

Flag	Coverage Δ
e2e	42.21% <0.00%> (-0.05%)	⬇️
experimental-e2e	11.82% <0.00%> (-39.76%)	⬇️
unit	53.84% <95.45%> (+0.02%)	⬆️

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.

[thetechnick]

Looks good from my end :)

[Copilot]

Pull request overview

Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

internal/operator-controller/controllers/clusterextensionrevision_controller.go:149

• The error returned from buildBoxcutterPhases is being re-wrapped with %v and an outdated message ("converting to boxcutter revision"). Consider updating the message to reflect that this step builds phases/options, and use %w so callers can unwrap the original error for classification/logging.

	phases, opts, err := c.buildBoxcutterPhases(ctx, cer)
	if err != nil {
		setRetryingConditions(cer, err.Error())
		return ctrl.Result{}, fmt.Errorf("converting to boxcutter revision: %v", err)
	}

---

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

[dtfranz]

Whoops, sorry for erasing then re-applying your changes @perdasilva 😵 I had a combo of the github page not showing your changes and my local branch having a diff from upstream so I just assumed I had forgotten to push something. The branch should be identical now to your push, though.