Skip to content

Commit 63fbe73

Browse files
JoelSpeedJoelSpeed
committed
Drop oVirt support
1 parent 191a28d commit 63fbe73

9 files changed

+0
-99
lines changed

‎README.md

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,8 +38,6 @@ Have a question? See our [Frequently Asked Questions](FAQ.md) for common inquiri
3838
- [cluster-api-provider-openstack](https://github.com/openshift/cluster-api-provider-openstack)
3939

4040
- [cluster-api-provider-baremetal](https://github.com/openshift/cluster-api-provider-baremetal)
41-
42-
- [cluster-api-provider-ovirt](https://github.com/openshift/cluster-api-provider-ovirt)
4341

4442
- [cluster-api-provider-ibmcloud](https://github.com/openshift/cluster-api-provider-ibmcloud)
4543

‎docs/dev/hacking-guide.md

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,6 @@ Machine API consists of a number different components:
4141
- https://github.com/openshift/machine-api-operator/tree/master/pkg/controller/vsphere
4242
- https://github.com/openshift/cluster-api-provider-openstack
4343
- https://github.com/openshift/cluster-api-provider-baremetal
44-
- https://github.com/openshift/cluster-api-provider-ovirt
4544

4645
### How to start contributing
4746

@@ -154,7 +153,6 @@ data:
154153
"clusterAPIControllerBareMetal": "quay.io/openshift-release-dev/ocp-v4.0-art-dev@sha256:...",
155154
"clusterAPIControllerAzure": "quay.io/openshift-release-dev/ocp-v4.0-art-dev@sha256:...",
156155
"clusterAPIControllerGCP": "quay.io/openshift-release-dev/ocp-v4.0-art-dev@sha256:...",
157-
"clusterAPIControllerOvirt": "quay.io/openshift-release-dev/ocp-v4.0-art-dev@sha256:...",
158156
"clusterAPIControllerVSphere": "quay.io/openshift-release-dev/ocp-v4.0-art-dev@sha256:...",
159157
"baremetalOperator": "quay.io/openshift-release-dev/ocp-v4.0-art-dev@sha256:...",
160158
"baremetalIronic": "quay.io/openshift-release-dev/ocp-v4.0-art-dev@sha256:...",

‎docs/user/machine-api-operator-overview.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,6 @@ Providers which currently works with MAO, are:
5353
- [vSphere](https://github.com/openshift/machine-api-operator/tree/master/pkg/controller/vsphere)
5454
- [Azure](https://github.com/openshift/cluster-api-provider-azure)
5555
- [BareMetal](https://github.com/openshift/cluster-api-provider-baremetal/)
56-
- [OVirt](https://github.com/openshift/cluster-api-provider-ovirt)
5756

5857
## Works closely, but not directly responsible for
5958

‎install/0000_30_machine-api-operator_00_credentials-request.yaml

Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -212,24 +212,6 @@ spec:
212212
---
213213
apiVersion: cloudcredential.openshift.io/v1
214214
kind: CredentialsRequest
215-
metadata:
216-
labels:
217-
controller-tools.k8s.io: "1.0"
218-
name: openshift-machine-api-ovirt
219-
namespace: openshift-cloud-credential-operator
220-
annotations:
221-
capability.openshift.io/name: MachineAPI+CloudCredential
222-
include.release.openshift.io/self-managed-high-availability: "true"
223-
spec:
224-
secretRef:
225-
name: ovirt-credentials
226-
namespace: openshift-machine-api
227-
providerSpec:
228-
apiVersion: cloudcredential.openshift.io/v1
229-
kind: OvirtProviderSpec
230-
---
231-
apiVersion: cloudcredential.openshift.io/v1
232-
kind: CredentialsRequest
233215
metadata:
234216
labels:
235217
controller-tools.k8s.io: "1.0"

‎install/0000_30_machine-api-operator_01_images.configmap.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@ data:
2222
"clusterAPIControllerGCP": "quay.io/openshift/origin-gcp-machine-controllers",
2323
"clusterAPIControllerIBMCloud": "quay.io/openshift/origin-ibmcloud-machine-controllers",
2424
"clusterAPIControllerNutanix": "quay.io/openshift/origin-nutanix-machine-controllers",
25-
"clusterAPIControllerOvirt": "quay.io/openshift/origin-ovirt-machine-controllers",
2625
"clusterAPIControllerPowerVS": "quay.io/openshift/origin-powervs-machine-controllers",
2726
"clusterAPIControllerVSphere": "quay.io/openshift/origin-machine-api-operator"
2827
}

‎install/image-references

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -46,10 +46,6 @@ spec:
4646
from:
4747
kind: DockerImage
4848
name: quay.io/openshift/origin-kube-rbac-proxy
49-
- name: ovirt-machine-controllers
50-
from:
51-
kind: DockerImage
52-
name: quay.io/openshift/origin-ovirt-machine-controllers
5349
- name: nutanix-machine-controllers
5450
from:
5551
kind: DockerImage

‎pkg/operator/config.go

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,6 @@ type Images struct {
4545
ClusterAPIControllerBareMetal string `json:"clusterAPIControllerBareMetal"`
4646
ClusterAPIControllerAzure string `json:"clusterAPIControllerAzure"`
4747
ClusterAPIControllerGCP string `json:"clusterAPIControllerGCP"`
48-
ClusterAPIControllerOvirt string `json:"clusterAPIControllerOvirt"`
4948
ClusterAPIControllerVSphere string `json:"clusterAPIControllerVSphere"`
5049
ClusterAPIControllerIBMCloud string `json:"clusterAPIControllerIBMCloud"`
5150
ClusterAPIControllerPowerVS string `json:"clusterAPIControllerPowerVS"`
@@ -89,8 +88,6 @@ func getProviderControllerFromImages(platform configv1.PlatformType, images Imag
8988
return images.ClusterAPIControllerGCP, nil
9089
case configv1.BareMetalPlatformType:
9190
return images.ClusterAPIControllerBareMetal, nil
92-
case configv1.OvirtPlatformType:
93-
return images.ClusterAPIControllerOvirt, nil
9491
case configv1.VSpherePlatformType:
9592
return images.ClusterAPIControllerVSphere, nil
9693
case configv1.IBMCloudPlatformType:

‎pkg/operator/config_test.go

Lines changed: 0 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@ const (
2121
expectedGCPImage = "quay.io/openshift/origin-gcp-machine-controllers"
2222
expectedLibvirtImage = "quay.io/openshift/origin-libvirt-machine-controllers"
2323
expectedOpenstackImage = "quay.io/openshift/origin-openstack-machine-api-provider"
24-
expectedOvirtImage = "quay.io/openshift/origin-ovirt-machine-controllers"
2524
expectedPowerVSImage = "quay.io/openshift/origin-powervs-machine-controllers"
2625
expectedVSphereImage = "quay.io/openshift/origin-machine-api-operator"
2726
expectedNutanixImage = "quay.io/openshift/origin-nutanix-machine-controllers"
@@ -165,22 +164,6 @@ func TestGetProviderFromInfrastructure(t *testing.T) {
165164
},
166165
},
167166
expected: configv1.NonePlatformType,
168-
}, {
169-
infra: &configv1.Infrastructure{
170-
Status: configv1.InfrastructureStatus{
171-
PlatformStatus: &configv1.PlatformStatus{
172-
Type: configv1.OvirtPlatformType,
173-
},
174-
},
175-
},
176-
expected: configv1.OvirtPlatformType,
177-
}, {
178-
infra: &configv1.Infrastructure{
179-
Status: configv1.InfrastructureStatus{
180-
Platform: configv1.OvirtPlatformType,
181-
},
182-
},
183-
expected: "",
184167
}, {
185168
infra: &configv1.Infrastructure{
186169
Status: configv1.InfrastructureStatus{
@@ -246,9 +229,6 @@ func TestGetImagesFromJSONFile(t *testing.T) {
246229
if img.ClusterAPIControllerGCP != expectedGCPImage {
247230
t.Errorf("failed getImagesFromJSONFile. Expected: %s, got: %s", expectedGCPImage, img.ClusterAPIControllerGCP)
248231
}
249-
if img.ClusterAPIControllerOvirt != expectedOvirtImage {
250-
t.Errorf("failed getImagesFromJSONFile. Expected: %s, got: %s", expectedOvirtImage, img.ClusterAPIControllerOvirt)
251-
}
252232
if img.ClusterAPIControllerVSphere != expectedVSphereImage {
253233
t.Errorf("failed getImagesFromJSONFile. Expected: %s, got: %s", expectedVSphereImage, img.ClusterAPIControllerVSphere)
254234
}
@@ -303,10 +283,6 @@ func TestGetProviderControllerFromImages(t *testing.T) {
303283
provider: configv1.NonePlatformType,
304284
expectedImage: clusterAPIControllerNoOp,
305285
},
306-
{
307-
provider: configv1.OvirtPlatformType,
308-
expectedImage: expectedOvirtImage,
309-
},
310286
{
311287
provider: configv1.PowerVSPlatformType,
312288
expectedImage: expectedPowerVSImage,
@@ -388,10 +364,6 @@ func TestGetTerminationHandlerFromImages(t *testing.T) {
388364
provider: configv1.NonePlatformType,
389365
expectedImage: clusterAPIControllerNoOp,
390366
},
391-
{
392-
provider: configv1.OvirtPlatformType,
393-
expectedImage: clusterAPIControllerNoOp,
394-
},
395367
{
396368
provider: configv1.PowerVSPlatformType,
397369
expectedImage: clusterAPIControllerNoOp,

‎pkg/operator/operator_test.go

Lines changed: 0 additions & 40 deletions
Original file line numberDiff line numberDiff line change
@@ -152,10 +152,6 @@ func TestOperatorSync_NoOp(t *testing.T) {
152152
platform: openshiftv1.VSpherePlatformType,
153153
expectedNoop: false,
154154
},
155-
{
156-
platform: openshiftv1.OvirtPlatformType,
157-
expectedNoop: false,
158-
},
159155
{
160156
platform: openshiftv1.PowerVSPlatformType,
161157
expectedNoop: false,
@@ -636,42 +632,6 @@ func TestMAOConfigFromInfrastructure(t *testing.T) {
636632
},
637633
},
638634
},
639-
{
640-
name: string(openshiftv1.OvirtPlatformType),
641-
platform: openshiftv1.OvirtPlatformType,
642-
infra: infra,
643-
featureGate: &openshiftv1.FeatureGate{
644-
ObjectMeta: metav1.ObjectMeta{
645-
Name: "cluster",
646-
},
647-
Status: openshiftv1.FeatureGateStatus{
648-
FeatureGates: []openshiftv1.FeatureGateDetails{
649-
{
650-
Version: "",
651-
Enabled: []openshiftv1.FeatureGateAttributes{{Name: apifeatures.FeatureGateMachineAPIMigration}},
652-
Disabled: []openshiftv1.FeatureGateAttributes{{Name: apifeatures.FeatureGateMachineAPIOperatorDisableMachineHealthCheckController}},
653-
},
654-
},
655-
},
656-
},
657-
proxy: proxy,
658-
expectedConfig: &OperatorConfig{
659-
TargetNamespace: targetNamespace,
660-
Proxy: proxy,
661-
Controllers: Controllers{
662-
Provider: images.ClusterAPIControllerOvirt,
663-
MachineSet: images.MachineAPIOperator,
664-
NodeLink: images.MachineAPIOperator,
665-
MachineHealthCheck: images.MachineAPIOperator,
666-
TerminationHandler: clusterAPIControllerNoOp,
667-
KubeRBACProxy: images.KubeRBACProxy,
668-
},
669-
PlatformType: openshiftv1.OvirtPlatformType,
670-
Features: map[string]bool{
671-
"MachineAPIMigration": true,
672-
},
673-
},
674-
},
675635
{
676636
name: string(openshiftv1.NonePlatformType),
677637
platform: openshiftv1.NonePlatformType,

0 commit comments

Comments
 (0)