test: cover migration acknowledgments and enforce dry-run eligibility

[tmshort]

Summary

Add fixture E2E coverage for seven soft compatibility checks and five non-overridable hard blocks. Each soft case verifies refusal without its acknowledgment, a non-mutating acknowledged dry-run, and successful conversion with the audit annotation. Each hard case verifies that acknowledgments cannot bypass the rejection.

Fix single-operator dry-run so it performs the same readiness, compatibility, and catalog checks as conversion. Add unit coverage for that behavior. The new migration/test-e2e-acknowledgments target runs the isolated fixture scenarios with coverage in CI. Update the migration E2E guide and test plan.

Validation

• make migration/test-unit
• go test -tags=e2e ./test/e2e/migration -run '^$'
• make migration/test-e2e-acknowledgments (all 12 scenarios passed before the branch was rebased; the rebase preserved the patch)

Tracks OPRUN-4762.

[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

[coderabbitai]

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 37 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 527451f7-5f9f-46ca-91af-d7c54a2d9bba

📥 Commits

Reviewing files that changed from the base of the PR and between 83e4e91 and b779602.

📒 Files selected for processing (7)

• .github/workflows/migration-test.yaml
• migration.mk
• migration/examples/cmd/migrate-operators-v0-to-v1/convert.go
• migration/examples/cmd/migrate-operators-v0-to-v1/dryrun_eligibility_test.go
• specs/20260821-migration-v0-to-v1/e2e.md
• specs/20260821-migration-v0-to-v1/test-plan.md
• test/e2e/migration/acknowledgment_test.go

• Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}