OCPBUGS-77941: Remove cluster-api-provider-libvirt references

[sdodson]

Summary by CodeRabbit

• Documentation

  • Removed Libvirt platform references from documentation and configuration files

• Chores

  • Removed Libvirt-related entries from image configurations and removed associated test coverage

[openshift-ci-robot]

@sdodson: This pull request references Jira Issue OCPBUGS-77941, which is valid. The bug has been moved to the POST state.

3 validation(s) were run on this bug

• bug is open, matching expected state (open)
• bug target version (4.22.0) matches configured target version for branch (4.22.0)
• bug is in the state New, which is one of the valid states (NEW, ASSIGNED, POST)

Requesting review from QA contact:
/cc @huali9

The bug has been updated to refer to the pull request using the external bug tracker.

Details

In response to this:

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository: openshift/coderabbit/.coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 72f81e90-9f83-4792-99ad-8c4600318f0b

📥 Commits

Reviewing files that changed from the base of the PR and between 8846555 and e74bb11.

📒 Files selected for processing (7)

• README.md
• docs/dev/hacking-guide.md
• install/0000_30_machine-api-operator_01_images.configmap.yaml
• install/image-references
• pkg/operator/config.go
• pkg/operator/config_test.go
• pkg/operator/operator_test.go

💤 Files with no reviewable changes (7)

• pkg/operator/operator_test.go
• pkg/operator/config_test.go
• README.md
• install/0000_30_machine-api-operator_01_images.configmap.yaml
• pkg/operator/config.go
• docs/dev/hacking-guide.md
• install/image-references

---

Walkthrough

This pull request removes all Libvirt-related support from the machine-api-operator codebase, including image references from ConfigMaps and image streams, the ClusterAPIControllerLibvirt struct field, and all corresponding Libvirt platform test cases across multiple test files.

Changes

Cohort / File(s)	Summary
Documentation and Configuration References README.md, docs/dev/hacking-guide.md, install/image-references, install/0000_30_machine-api-operator_01_images.configmap.yaml	Removed all Libvirt-related entries from documentation, ConfigMap images object, and image stream references including the libvirt-machine-controllers DockerImage.
Configuration Code pkg/operator/config.go	Removed ClusterAPIControllerLibvirt field from the Images struct and eliminated the Libvirt platform case from getProviderControllerFromImages function.
Test Files pkg/operator/config_test.go, pkg/operator/operator_test.go	Removed all Libvirt-related test cases and assertions including expectedLibvirtImage constant and LibvirtPlatformType test paths from multiple test functions.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'OCPBUGS-77941: Remove cluster-api-provider-libvirt references' directly and clearly summarizes the main change: systematic removal of libvirt-related references across documentation, configuration, and test files.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Stable And Deterministic Test Names	✅ Passed	The PR removes Libvirt test cases from traditional Go test functions with static, deterministic names containing no dynamic content like timestamps, UUIDs, or generated identifiers.
Test Structure And Quality	✅ Passed	PR modifies standard Go unit tests (not Ginkgo tests), maintaining proper cleanup, meaningful error messages, and appropriate timeouts without using Ginkgo patterns.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.

---

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

[openshift-ci-robot]

@sdodson: This pull request references Jira Issue OCPBUGS-77941, which is valid.

3 validation(s) were run on this bug

• bug is open, matching expected state (open)
• bug target version (4.22.0) matches configured target version for branch (4.22.0)
• bug is in the state POST, which is one of the valid states (NEW, ASSIGNED, POST)

Requesting review from QA contact:
/cc @huali9

Details

In response to this:

Summary by CodeRabbit

• Documentation

• Removed Libvirt platform references from documentation and configuration files

• Chores

• Removed Libvirt-related entries from image configurations and removed associated test coverage

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

[sdodson]

@JoelSpeed following after your pattern in ovirt removal here, i notice the platform type was not removed from openshift/api I assume it can never be removed there?

[sdodson]

/jira backport release-4.21

[openshift-ci-robot]

@sdodson: The following backport issues have been created:

• OCPBUGS-77943 for branch release-4.21

Queuing cherrypicks to the requested branches to be created after this PR merges:
/cherrypick release-4.21

Details

In response to this:

/jira backport release-4.21

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

[openshift-cherrypick-robot]

@openshift-ci-robot: once the present PR merges, I will cherry-pick it on top of release-4.21 in a new PR and assign it to you.

Details

In response to this:

@sdodson: The following backport issues have been created:

• OCPBUGS-77943 for branch release-4.21

Queuing cherrypicks to the requested branches to be created after this PR merges:
/cherrypick release-4.21

In response to this:

/jira backport release-4.21

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

[JoelSpeed]

/lgtm
/approve

I assume it can never be removed there?

I think we can probably do some cleanup there as we hit 5.0, since we should be confident no one is upgrading into 5.0 on the various platforms no longer supported. Alibaba too needs cleanup

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: JoelSpeed

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

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [JoelSpeed]

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

[sdodson]

I'll leave this for pre-merge verification for a few days but plan to move forward with this within the next few days if that's not done.

[huali9]

/verified by @huali9

Checked job ci/prow/e2e-aws-operator passed and machine-api-operator works well, no libvirt error in the pod.

[openshift-ci-robot]

@huali9: This PR has been marked as verified by @huali9.

Details

In response to this:

/verified by @huali9

Checked job ci/prow/e2e-aws-operator passed and machine-api-operator works well, no libvirt error in the pod.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

[JoelSpeed]

/override ci/prow/e2e-metal-ipi-ovn-ipv6

Confident the failures aren't related to this

[openshift-ci]

@JoelSpeed: Overrode contexts on behalf of JoelSpeed: ci/prow/e2e-metal-ipi-ovn-ipv6

Details

In response to this:

/override ci/prow/e2e-metal-ipi-ovn-ipv6

Confident the failures aren't related to this

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

[openshift-ci]

@sdodson: all tests passed!

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

[openshift-ci-robot]

@sdodson: Jira Issue OCPBUGS-77941: Some pull requests linked via external trackers have merged:

• openshift/machine-api-operator#1472

The following pull request, linked via external tracker, has not merged:

• openshift/release#75837 is open

All associated pull requests must be merged or unlinked from the Jira bug in order for it to move to the next state. Once unlinked, request a bug refresh with /jira refresh.

Jira Issue OCPBUGS-77941 has not been moved to the MODIFIED state.

This PR is marked as verified. If the remaining PRs listed above are marked as verified before merging, the issue will automatically be moved to VERIFIED after all of the changes from the PRs are available in an accepted nightly payload.

Details

In response to this:

Summary by CodeRabbit

• Documentation

• Removed Libvirt platform references from documentation and configuration files

• Chores

• Removed Libvirt-related entries from image configurations and removed associated test coverage

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

[openshift-cherrypick-robot]

@openshift-ci-robot: new pull request created: #1473

Details

In response to this:

@sdodson: The following backport issues have been created:

• OCPBUGS-77943 for branch release-4.21

Queuing cherrypicks to the requested branches to be created after this PR merges:
/cherrypick release-4.21

In response to this:

/jira backport release-4.21

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.