Modified parent logic to not check size, but only changes in existence.

Author: vr4manta

config/v1/types_infrastructure.go

@@ -295,9 +295,8 @@ type ExternalPlatformSpec struct {
 // PlatformSpec holds the desired state specific to the underlying infrastructure provider
 // of the current cluster. Since these are used at spec-level for the underlying cluster, it
 // is supposed that only one of the spec structs is set.
-// +openshift:validation:FeatureGateAwareXValidation:featureGate="",rule="!has(oldSelf.vsphere) && has(self.vsphere) && has(self.vsphere.vcenters) ? size(self.vsphere.vcenters) < 2 : true",message="vcenters can have at most 1 item when configured post-install"
-// +openshift:validation:FeatureGateAwareXValidation:featureGate=VSphereMultiVCenterDay2,rule="has(oldSelf.vsphere) && has(oldSelf.vsphere.vcenters) ? (has(self.vsphere) && has(self.vsphere.vcenters) && size(self.vsphere.vcenters) > 0) : true",message="vcenters is required once set and must have at least 1 item"
-// +openshift:validation:FeatureGateAwareXValidation:featureGate=VSphereMultiVCenterDay2,rule="!has(oldSelf.vsphere) && has(self.vsphere) && has(self.vsphere.vcenters) ? size(self.vsphere.vcenters) > 0 : true",message="vcenters must have at least 1 item when initially configured"
+// +openshift:validation:FeatureGateAwareXValidation:featureGate="",rule="!has(oldSelf.vsphere) && has(self.vsphere) ? (has(self.vsphere.vcenters) && size(self.vsphere.vcenters) < 2) : true",message="vcenters can have at most 1 item when configured post-install"
+// +openshift:validation:FeatureGateAwareXValidation:featureGate=VSphereMultiVCenterDay2,rule="has(oldSelf.vsphere) && has(oldSelf.vsphere.vcenters) ? (has(self.vsphere) && has(self.vsphere.vcenters)) : true",message="vcenters is required once set and cannot be removed"
 type PlatformSpec struct {
 	// type is the underlying infrastructure provider for the cluster. This
 	// value controls whether infrastructure automation such as service load
@@ -1645,10 +1644,11 @@ type VSpherePlatformNodeNetworking struct {
 // +kubebuilder:validation:XValidation:rule="!has(oldSelf.ingressIPs) || has(self.ingressIPs)",message="ingressIPs list is required once set"
 type VSpherePlatformSpec struct {
 	// vcenters holds the connection details for services to communicate with vCenter.
-	// Currently, only a single vCenter is supported, but in tech preview 3 vCenters are supported.
+	// Currently, up to 3 vCenters are supported.
 	// Once the cluster has been installed, you are unable to change the current number of defined
-	// vCenters except in the case where the cluster has been upgraded from a version of OpenShift
-	// where the vsphere platform spec was not present.  You may make modifications to the existing
+	// vCenters except when 1.) the cluster has been upgraded from a version of OpenShift
+	// where the vsphere platform spec was not present or 2.) in TechPreview you are able to add and
+	// remove vCenters but may not remove all vCenters.  You may make modifications to the existing
 	// vCenters that are defined in the vcenters list in order to match with any added or modified
 	// failure domains.
 	// ---

config/v1/zz_generated.crd-manifests/0000_10_config-operator_01_infrastructures-CustomNoUpgrade.crd.yaml

@@ -1031,10 +1031,11 @@ spec:
                       vcenters:
                         description: |-
                           vcenters holds the connection details for services to communicate with vCenter.
-                          Currently, only a single vCenter is supported, but in tech preview 3 vCenters are supported.
+                          Currently, up to 3 vCenters are supported.
                           Once the cluster has been installed, you are unable to change the current number of defined
-                          vCenters except in the case where the cluster has been upgraded from a version of OpenShift
-                          where the vsphere platform spec was not present.  You may make modifications to the existing
+                          vCenters except when 1.) the cluster has been upgraded from a version of OpenShift
+                          where the vsphere platform spec was not present or 2.) in TechPreview you are able to add and
+                          remove vCenters but may not remove all vCenters.  You may make modifications to the existing
                           vCenters that are defined in the vcenters list in order to match with any added or modified
                           failure domains.
                         items:
@@ -1093,14 +1094,9 @@ spec:
                       rule: '!has(oldSelf.ingressIPs) || has(self.ingressIPs)'
                 type: object
                 x-kubernetes-validations:
-                - message: vcenters is required once set and must have at least 1
-                    item
+                - message: vcenters is required once set and cannot be removed
                   rule: 'has(oldSelf.vsphere) && has(oldSelf.vsphere.vcenters) ? (has(self.vsphere)
-                    && has(self.vsphere.vcenters) && size(self.vsphere.vcenters) >
-                    0) : true'
-                - message: vcenters must have at least 1 item when initially configured
-                  rule: '!has(oldSelf.vsphere) && has(self.vsphere) && has(self.vsphere.vcenters)
-                    ? size(self.vsphere.vcenters) > 0 : true'
+                    && has(self.vsphere.vcenters)) : true'
             type: object
           status:
             description: status holds observed values from the cluster. They may not

config/v1/zz_generated.crd-manifests/0000_10_config-operator_01_infrastructures-Default.crd.yaml

@@ -965,10 +965,11 @@ spec:
                       vcenters:
                         description: |-
                           vcenters holds the connection details for services to communicate with vCenter.
-                          Currently, only a single vCenter is supported, but in tech preview 3 vCenters are supported.
+                          Currently, up to 3 vCenters are supported.
                           Once the cluster has been installed, you are unable to change the current number of defined
-                          vCenters except in the case where the cluster has been upgraded from a version of OpenShift
-                          where the vsphere platform spec was not present.  You may make modifications to the existing
+                          vCenters except when 1.) the cluster has been upgraded from a version of OpenShift
+                          where the vsphere platform spec was not present or 2.) in TechPreview you are able to add and
+                          remove vCenters but may not remove all vCenters.  You may make modifications to the existing
                           vCenters that are defined in the vcenters list in order to match with any added or modified
                           failure domains.
                         items:
@@ -1029,8 +1030,8 @@ spec:
                 type: object
                 x-kubernetes-validations:
                 - message: vcenters can have at most 1 item when configured post-install
-                  rule: '!has(oldSelf.vsphere) && has(self.vsphere) && has(self.vsphere.vcenters)
-                    ? size(self.vsphere.vcenters) < 2 : true'
+                  rule: '!has(oldSelf.vsphere) && has(self.vsphere) ? (has(self.vsphere.vcenters)
+                    && size(self.vsphere.vcenters) < 2) : true'
             type: object
           status:
             description: status holds observed values from the cluster. They may not

config/v1/zz_generated.crd-manifests/0000_10_config-operator_01_infrastructures-DevPreviewNoUpgrade.crd.yaml

@@ -1031,10 +1031,11 @@ spec:
                       vcenters:
                         description: |-
                           vcenters holds the connection details for services to communicate with vCenter.
-                          Currently, only a single vCenter is supported, but in tech preview 3 vCenters are supported.
+                          Currently, up to 3 vCenters are supported.
                           Once the cluster has been installed, you are unable to change the current number of defined
-                          vCenters except in the case where the cluster has been upgraded from a version of OpenShift
-                          where the vsphere platform spec was not present.  You may make modifications to the existing
+                          vCenters except when 1.) the cluster has been upgraded from a version of OpenShift
+                          where the vsphere platform spec was not present or 2.) in TechPreview you are able to add and
+                          remove vCenters but may not remove all vCenters.  You may make modifications to the existing
                           vCenters that are defined in the vcenters list in order to match with any added or modified
                           failure domains.
                         items:
@@ -1093,14 +1094,9 @@ spec:
                       rule: '!has(oldSelf.ingressIPs) || has(self.ingressIPs)'
                 type: object
                 x-kubernetes-validations:
-                - message: vcenters is required once set and must have at least 1
-                    item
+                - message: vcenters is required once set and cannot be removed
                   rule: 'has(oldSelf.vsphere) && has(oldSelf.vsphere.vcenters) ? (has(self.vsphere)
-                    && has(self.vsphere.vcenters) && size(self.vsphere.vcenters) >
-                    0) : true'
-                - message: vcenters must have at least 1 item when initially configured
-                  rule: '!has(oldSelf.vsphere) && has(self.vsphere) && has(self.vsphere.vcenters)
-                    ? size(self.vsphere.vcenters) > 0 : true'
+                    && has(self.vsphere.vcenters)) : true'
             type: object
           status:
             description: status holds observed values from the cluster. They may not

config/v1/zz_generated.crd-manifests/0000_10_config-operator_01_infrastructures-OKD.crd.yaml

@@ -965,10 +965,11 @@ spec:
                       vcenters:
                         description: |-
                           vcenters holds the connection details for services to communicate with vCenter.
-                          Currently, only a single vCenter is supported, but in tech preview 3 vCenters are supported.
+                          Currently, up to 3 vCenters are supported.
                           Once the cluster has been installed, you are unable to change the current number of defined
-                          vCenters except in the case where the cluster has been upgraded from a version of OpenShift
-                          where the vsphere platform spec was not present.  You may make modifications to the existing
+                          vCenters except when 1.) the cluster has been upgraded from a version of OpenShift
+                          where the vsphere platform spec was not present or 2.) in TechPreview you are able to add and
+                          remove vCenters but may not remove all vCenters.  You may make modifications to the existing
                           vCenters that are defined in the vcenters list in order to match with any added or modified
                           failure domains.
                         items:
@@ -1029,8 +1030,8 @@ spec:
                 type: object
                 x-kubernetes-validations:
                 - message: vcenters can have at most 1 item when configured post-install
-                  rule: '!has(oldSelf.vsphere) && has(self.vsphere) && has(self.vsphere.vcenters)
-                    ? size(self.vsphere.vcenters) < 2 : true'
+                  rule: '!has(oldSelf.vsphere) && has(self.vsphere) ? (has(self.vsphere.vcenters)
+                    && size(self.vsphere.vcenters) < 2) : true'
             type: object
           status:
             description: status holds observed values from the cluster. They may not

config/v1/zz_generated.crd-manifests/0000_10_config-operator_01_infrastructures-TechPreviewNoUpgrade.crd.yaml

@@ -1031,10 +1031,11 @@ spec:
                       vcenters:
                         description: |-
                           vcenters holds the connection details for services to communicate with vCenter.
-                          Currently, only a single vCenter is supported, but in tech preview 3 vCenters are supported.
+                          Currently, up to 3 vCenters are supported.
                           Once the cluster has been installed, you are unable to change the current number of defined
-                          vCenters except in the case where the cluster has been upgraded from a version of OpenShift
-                          where the vsphere platform spec was not present.  You may make modifications to the existing
+                          vCenters except when 1.) the cluster has been upgraded from a version of OpenShift
+                          where the vsphere platform spec was not present or 2.) in TechPreview you are able to add and
+                          remove vCenters but may not remove all vCenters.  You may make modifications to the existing
                           vCenters that are defined in the vcenters list in order to match with any added or modified
                           failure domains.
                         items:
@@ -1095,8 +1096,8 @@ spec:
                 type: object
                 x-kubernetes-validations:
                 - message: vcenters can have at most 1 item when configured post-install
-                  rule: '!has(oldSelf.vsphere) && has(self.vsphere) && has(self.vsphere.vcenters)
-                    ? size(self.vsphere.vcenters) < 2 : true'
+                  rule: '!has(oldSelf.vsphere) && has(self.vsphere) ? (has(self.vsphere.vcenters)
+                    && size(self.vsphere.vcenters) < 2) : true'
             type: object
           status:
             description: status holds observed values from the cluster. They may not

config/v1/zz_generated.featuregated-crd-manifests/infrastructures.config.openshift.io/AAA_ungated.yaml

@@ -946,10 +946,11 @@ spec:
                       vcenters:
                         description: |-
                           vcenters holds the connection details for services to communicate with vCenter.
-                          Currently, only a single vCenter is supported, but in tech preview 3 vCenters are supported.
+                          Currently, up to 3 vCenters are supported.
                           Once the cluster has been installed, you are unable to change the current number of defined
-                          vCenters except in the case where the cluster has been upgraded from a version of OpenShift
-                          where the vsphere platform spec was not present.  You may make modifications to the existing
+                          vCenters except when 1.) the cluster has been upgraded from a version of OpenShift
+                          where the vsphere platform spec was not present or 2.) in TechPreview you are able to add and
+                          remove vCenters but may not remove all vCenters.  You may make modifications to the existing
                           vCenters that are defined in the vcenters list in order to match with any added or modified
                           failure domains.
                         items:
@@ -1006,8 +1007,8 @@ spec:
                 type: object
                 x-kubernetes-validations:
                 - message: vcenters can have at most 1 item when configured post-install
-                  rule: '!has(oldSelf.vsphere) && has(self.vsphere) && has(self.vsphere.vcenters)
-                    ? size(self.vsphere.vcenters) < 2 : true'
+                  rule: '!has(oldSelf.vsphere) && has(self.vsphere) ? (has(self.vsphere.vcenters)
+                    && size(self.vsphere.vcenters) < 2) : true'
             type: object
           status:
             description: status holds observed values from the cluster. They may not

config/v1/zz_generated.featuregated-crd-manifests/infrastructures.config.openshift.io/AWSClusterHostedDNSInstall.yaml

@@ -944,10 +944,11 @@ spec:
                       vcenters:
                         description: |-
                           vcenters holds the connection details for services to communicate with vCenter.
-                          Currently, only a single vCenter is supported, but in tech preview 3 vCenters are supported.
+                          Currently, up to 3 vCenters are supported.
                           Once the cluster has been installed, you are unable to change the current number of defined
-                          vCenters except in the case where the cluster has been upgraded from a version of OpenShift
-                          where the vsphere platform spec was not present.  You may make modifications to the existing
+                          vCenters except when 1.) the cluster has been upgraded from a version of OpenShift
+                          where the vsphere platform spec was not present or 2.) in TechPreview you are able to add and
+                          remove vCenters but may not remove all vCenters.  You may make modifications to the existing
                           vCenters that are defined in the vcenters list in order to match with any added or modified
                           failure domains.
                         items:

config/v1/zz_generated.featuregated-crd-manifests/infrastructures.config.openshift.io/AWSDualStackInstall.yaml

@@ -944,10 +944,11 @@ spec:
                       vcenters:
                         description: |-
                           vcenters holds the connection details for services to communicate with vCenter.
-                          Currently, only a single vCenter is supported, but in tech preview 3 vCenters are supported.
+                          Currently, up to 3 vCenters are supported.
                           Once the cluster has been installed, you are unable to change the current number of defined
-                          vCenters except in the case where the cluster has been upgraded from a version of OpenShift
-                          where the vsphere platform spec was not present.  You may make modifications to the existing
+                          vCenters except when 1.) the cluster has been upgraded from a version of OpenShift
+                          where the vsphere platform spec was not present or 2.) in TechPreview you are able to add and
+                          remove vCenters but may not remove all vCenters.  You may make modifications to the existing
                           vCenters that are defined in the vcenters list in order to match with any added or modified
                           failure domains.
                         items:

config/v1/zz_generated.featuregated-crd-manifests/infrastructures.config.openshift.io/AzureClusterHostedDNSInstall.yaml

@@ -944,10 +944,11 @@ spec:
                       vcenters:
                         description: |-
                           vcenters holds the connection details for services to communicate with vCenter.
-                          Currently, only a single vCenter is supported, but in tech preview 3 vCenters are supported.
+                          Currently, up to 3 vCenters are supported.
                           Once the cluster has been installed, you are unable to change the current number of defined
-                          vCenters except in the case where the cluster has been upgraded from a version of OpenShift
-                          where the vsphere platform spec was not present.  You may make modifications to the existing
+                          vCenters except when 1.) the cluster has been upgraded from a version of OpenShift
+                          where the vsphere platform spec was not present or 2.) in TechPreview you are able to add and
+                          remove vCenters but may not remove all vCenters.  You may make modifications to the existing
                           vCenters that are defined in the vcenters list in order to match with any added or modified
                           failure domains.
                         items: