Begin using KubeVirt as the infrastructure platform instead of None#1282
Begin using KubeVirt as the infrastructure platform instead of None#1282openshift-merge-robot merged 1 commit intoopenshift:mainfrom
Conversation
Signed-off-by: David Vossel <davidvossel@gmail.com>
|
@davidvossel: The following tests failed, say
Full PR test history. Your PR dashboard. DetailsInstructions 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/test-infra repository. I understand the commands that are listed here. |
|
/approve |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: davidvossel, enxebre The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
The machine-config-operator had some left over logic from the old KubeVirt IPI effort that broke the KubeVirt Hypershift platform. This caused us to have to temporarily report the
Noneplatform on the infrastructure object. Otherwise we'd get a bunch of coredns and other networking related changes layered in that are no longer relevant for kubevirt+hypershift.I updated the mco here to strip out the old KubeVirt IPI logic but leave the KubeVirt platform intact. Now we can remove the weird workaround where we used
none. This is important because it's needed for us to begin integrating logic into the cso for the kubevirt-csi driver install.