ClusterNetworkOperator API: remove additionalRoutingCapabilities gate

fedepaol · fedepaol · commit 8182d27f78c3 · 2024-11-06T18:23:10.000+01:00
Removing the feature gate as the deployed daemonset is being used by
metallb and covered by MetalLB tests.

Signed-off-by: Federico Paolinelli &lt;fpaoline@redhat.com&gt;
diff --git a/features.md b/features.md
@@ -6,9 +6,9 @@
 | MachineAPIMigration| | | | | |  |
 | MachineAPIOperatorDisableMachineHealthCheckController| | | | | |  |
 | MultiArchInstallAzure| | | | | |  |
+| AdditionalRoutingCapabilities| <span style="background-color: #519450">Enabled</span> | <span style="background-color: #519450">Enabled</span> | | | |  |
 | GatewayAPI| | | <span style="background-color: #519450">Enabled</span> | <span style="background-color: #519450">Enabled</span> | |  |
 | AWSClusterHostedDNS| | | <span style="background-color: #519450">Enabled</span> | <span style="background-color: #519450">Enabled</span> | <span style="background-color: #519450">Enabled</span> | <span style="background-color: #519450">Enabled</span>  |
-| AdditionalRoutingCapabilities| | | <span style="background-color: #519450">Enabled</span> | <span style="background-color: #519450">Enabled</span> | <span style="background-color: #519450">Enabled</span> | <span style="background-color: #519450">Enabled</span>  |
 | AutomatedEtcdBackup| | | <span style="background-color: #519450">Enabled</span> | <span style="background-color: #519450">Enabled</span> | <span style="background-color: #519450">Enabled</span> | <span style="background-color: #519450">Enabled</span>  |
 | BootcNodeManagement| | | <span style="background-color: #519450">Enabled</span> | <span style="background-color: #519450">Enabled</span> | <span style="background-color: #519450">Enabled</span> | <span style="background-color: #519450">Enabled</span>  |
 | CSIDriverSharedResource| | | <span style="background-color: #519450">Enabled</span> | <span style="background-color: #519450">Enabled</span> | <span style="background-color: #519450">Enabled</span> | <span style="background-color: #519450">Enabled</span>  |
diff --git a/features/features.go b/features/features.go
@@ -240,7 +240,7 @@ var (
 							contactPerson("jcaamano").
 							productScope(ocpSpecific).
 							enhancementPR(legacyFeatureGateWithoutEnhancement).
-							enableIn(configv1.DevPreviewNoUpgrade, configv1.TechPreviewNoUpgrade).
+							enableIn(configv1.Default).
 							mustRegister()
 
 	FeatureGateRouteAdvertisements = newFeatureGate("RouteAdvertisements").
diff --git a/operator/v1/types_network.go b/operator/v1/types_network.go
@@ -54,7 +54,7 @@ type NetworkList struct {
 
 // NetworkSpec is the top-level network configuration object.
 // +kubebuilder:validation:XValidation:rule="!has(self.defaultNetwork) || !has(self.defaultNetwork.ovnKubernetesConfig) || !has(self.defaultNetwork.ovnKubernetesConfig.gatewayConfig) || !has(self.defaultNetwork.ovnKubernetesConfig.gatewayConfig.ipForwarding) || self.defaultNetwork.ovnKubernetesConfig.gatewayConfig.ipForwarding == oldSelf.defaultNetwork.ovnKubernetesConfig.gatewayConfig.ipForwarding || self.defaultNetwork.ovnKubernetesConfig.gatewayConfig.ipForwarding == 'Restricted' || self.defaultNetwork.ovnKubernetesConfig.gatewayConfig.ipForwarding == 'Global'",message="invalid value for IPForwarding, valid values are 'Restricted' or 'Global'"
-// +openshift:validation:FeatureGateAwareXValidation:featureGate=AdditionalRoutingCapabilities,rule="(has(self.additionalRoutingCapabilities) && ('FRR' in self.additionalRoutingCapabilities.providers)) || !has(self.defaultNetwork) || !has(self.defaultNetwork.ovnKubernetesConfig) || !has(self.defaultNetwork.ovnKubernetesConfig.routeAdvertisements) || self.defaultNetwork.ovnKubernetesConfig.routeAdvertisements != 'Enabled'",message="Route advertisements cannot be Enabled if 'FRR' routing capability provider is not available"
+// +openshift:validation:FeatureGateAwareXValidation:featureGate=RouteAdvertisements,rule="(has(self.additionalRoutingCapabilities) && ('FRR' in self.additionalRoutingCapabilities.providers)) || !has(self.defaultNetwork) || !has(self.defaultNetwork.ovnKubernetesConfig) || !has(self.defaultNetwork.ovnKubernetesConfig.routeAdvertisements) || self.defaultNetwork.ovnKubernetesConfig.routeAdvertisements != 'Enabled'",message="Route advertisements cannot be Enabled if 'FRR' routing capability provider is not available"
 type NetworkSpec struct {
 	OperatorSpec `json:",inline"`
 
diff --git a/operator/v1/zz_generated.crd-manifests/0000_70_network_01_networks-Default.crd.yaml b/operator/v1/zz_generated.crd-manifests/0000_70_network_01_networks-Default.crd.yaml
@@ -178,6 +178,39 @@ spec:
                 x-kubernetes-list-map-keys:
                 - name
                 x-kubernetes-list-type: map
+              additionalRoutingCapabilities:
+                description: |-
+                  additionalRoutingCapabilities describes components and relevant
+                  configuration providing additional routing capabilities. When set, it
+                  enables such components and the usage of the routing capabilities they
+                  provide for the machine network. Upstream operators, like MetalLB
+                  operator, requiring these capabilities may rely on, or automatically set
+                  this attribute. Network plugins may leverage advanced routing
+                  capabilities acquired through the enablement of these components but may
+                  require specific configuration on their side to do so; refer to their
+                  respective documentation and configuration options.
+                properties:
+                  providers:
+                    description: |-
+                      providers is a set of enabled components that provide additional routing
+                      capabilities. Entries on this list must be unique. The  only valid value
+                      is currrently "FRR" which provides FRR routing capabilities through the
+                      deployment of FRR.
+                    items:
+                      description: RoutingCapabilitiesProvider is a component providing
+                        routing capabilities.
+                      enum:
+                      - FRR
+                      type: string
+                    maxItems: 1
+                    minItems: 1
+                    type: array
+                    x-kubernetes-list-type: atomic
+                    x-kubernetes-validations:
+                    - rule: self.all(x, self.exists_one(y, x == y))
+                required:
+                - providers
+                type: object
               clusterNetwork:
                 description: |-
                   clusterNetwork is the IP address pool to use for pod IPs.
diff --git a/operator/v1/zz_generated.crd-manifests/0000_70_network_01_networks-DevPreviewNoUpgrade.crd.yaml b/operator/v1/zz_generated.crd-manifests/0000_70_network_01_networks-DevPreviewNoUpgrade.crd.yaml
@@ -178,39 +178,6 @@ spec:
                 x-kubernetes-list-map-keys:
                 - name
                 x-kubernetes-list-type: map
-              additionalRoutingCapabilities:
-                description: |-
-                  additionalRoutingCapabilities describes components and relevant
-                  configuration providing additional routing capabilities. When set, it
-                  enables such components and the usage of the routing capabilities they
-                  provide for the machine network. Upstream operators, like MetalLB
-                  operator, requiring these capabilities may rely on, or automatically set
-                  this attribute. Network plugins may leverage advanced routing
-                  capabilities acquired through the enablement of these components but may
-                  require specific configuration on their side to do so; refer to their
-                  respective documentation and configuration options.
-                properties:
-                  providers:
-                    description: |-
-                      providers is a set of enabled components that provide additional routing
-                      capabilities. Entries on this list must be unique. The  only valid value
-                      is currrently "FRR" which provides FRR routing capabilities through the
-                      deployment of FRR.
-                    items:
-                      description: RoutingCapabilitiesProvider is a component providing
-                        routing capabilities.
-                      enum:
-                      - FRR
-                      type: string
-                    maxItems: 1
-                    minItems: 1
-                    type: array
-                    x-kubernetes-list-type: atomic
-                    x-kubernetes-validations:
-                    - rule: self.all(x, self.exists_one(y, x == y))
-                required:
-                - providers
-                type: object
               clusterNetwork:
                 description: |-
                   clusterNetwork is the IP address pool to use for pod IPs.
diff --git a/operator/v1/zz_generated.crd-manifests/0000_70_network_01_networks-TechPreviewNoUpgrade.crd.yaml b/operator/v1/zz_generated.crd-manifests/0000_70_network_01_networks-TechPreviewNoUpgrade.crd.yaml
@@ -178,39 +178,6 @@ spec:
                 x-kubernetes-list-map-keys:
                 - name
                 x-kubernetes-list-type: map
-              additionalRoutingCapabilities:
-                description: |-
-                  additionalRoutingCapabilities describes components and relevant
-                  configuration providing additional routing capabilities. When set, it
-                  enables such components and the usage of the routing capabilities they
-                  provide for the machine network. Upstream operators, like MetalLB
-                  operator, requiring these capabilities may rely on, or automatically set
-                  this attribute. Network plugins may leverage advanced routing
-                  capabilities acquired through the enablement of these components but may
-                  require specific configuration on their side to do so; refer to their
-                  respective documentation and configuration options.
-                properties:
-                  providers:
-                    description: |-
-                      providers is a set of enabled components that provide additional routing
-                      capabilities. Entries on this list must be unique. The  only valid value
-                      is currrently "FRR" which provides FRR routing capabilities through the
-                      deployment of FRR.
-                    items:
-                      description: RoutingCapabilitiesProvider is a component providing
-                        routing capabilities.
-                      enum:
-                      - FRR
-                      type: string
-                    maxItems: 1
-                    minItems: 1
-                    type: array
-                    x-kubernetes-list-type: atomic
-                    x-kubernetes-validations:
-                    - rule: self.all(x, self.exists_one(y, x == y))
-                required:
-                - providers
-                type: object
               clusterNetwork:
                 description: |-
                   clusterNetwork is the IP address pool to use for pod IPs.
diff --git a/operator/v1/zz_generated.featuregated-crd-manifests/networks.operator.openshift.io/AdditionalRoutingCapabilities.yaml b/operator/v1/zz_generated.featuregated-crd-manifests/networks.operator.openshift.io/AdditionalRoutingCapabilities.yaml
@@ -874,13 +874,6 @@ spec:
                 type: boolean
             type: object
             x-kubernetes-validations:
-            - message: Route advertisements cannot be Enabled if 'FRR' routing capability
-                provider is not available
-              rule: (has(self.additionalRoutingCapabilities) && ('FRR' in self.additionalRoutingCapabilities.providers))
-                || !has(self.defaultNetwork) || !has(self.defaultNetwork.ovnKubernetesConfig)
-                || !has(self.defaultNetwork.ovnKubernetesConfig.routeAdvertisements)
-                || self.defaultNetwork.ovnKubernetesConfig.routeAdvertisements !=
-                'Enabled'
             - message: invalid value for IPForwarding, valid values are 'Restricted'
                 or 'Global'
               rule: '!has(self.defaultNetwork) || !has(self.defaultNetwork.ovnKubernetesConfig)
diff --git a/operator/v1/zz_generated.featuregated-crd-manifests/networks.operator.openshift.io/RouteAdvertisements.yaml b/operator/v1/zz_generated.featuregated-crd-manifests/networks.operator.openshift.io/RouteAdvertisements.yaml
@@ -857,6 +857,13 @@ spec:
                 type: boolean
             type: object
             x-kubernetes-validations:
+            - message: Route advertisements cannot be Enabled if 'FRR' routing capability
+                provider is not available
+              rule: (has(self.additionalRoutingCapabilities) && ('FRR' in self.additionalRoutingCapabilities.providers))
+                || !has(self.defaultNetwork) || !has(self.defaultNetwork.ovnKubernetesConfig)
+                || !has(self.defaultNetwork.ovnKubernetesConfig.routeAdvertisements)
+                || self.defaultNetwork.ovnKubernetesConfig.routeAdvertisements !=
+                'Enabled'
             - message: invalid value for IPForwarding, valid values are 'Restricted'
                 or 'Global'
               rule: '!has(self.defaultNetwork) || !has(self.defaultNetwork.ovnKubernetesConfig)
diff --git a/payload-manifests/featuregates/featureGate-Hypershift-Default.yaml b/payload-manifests/featuregates/featureGate-Hypershift-Default.yaml
@@ -16,9 +16,6 @@
                     {
                         "name": "AWSClusterHostedDNS"
                     },
-                    {
-                        "name": "AdditionalRoutingCapabilities"
-                    },
                     {
                         "name": "AutomatedEtcdBackup"
                     },
@@ -177,6 +174,9 @@
                     {
                         "name": "AWSEFSDriverVolumeMetrics"
                     },
+                    {
+                        "name": "AdditionalRoutingCapabilities"
+                    },
                     {
                         "name": "AdminNetworkPolicy"
                     },
diff --git a/payload-manifests/featuregates/featureGate-Hypershift-DevPreviewNoUpgrade.yaml b/payload-manifests/featuregates/featureGate-Hypershift-DevPreviewNoUpgrade.yaml
@@ -15,6 +15,9 @@
         "featureGates": [
             {
                 "disabled": [
+                    {
+                        "name": "AdditionalRoutingCapabilities"
+                    },
                     {
                         "name": "ClusterAPIInstall"
                     },
@@ -41,9 +44,6 @@
                     {
                         "name": "AWSEFSDriverVolumeMetrics"
                     },
-                    {
-                        "name": "AdditionalRoutingCapabilities"
-                    },
                     {
                         "name": "AdminNetworkPolicy"
                     },
diff --git a/payload-manifests/featuregates/featureGate-Hypershift-TechPreviewNoUpgrade.yaml b/payload-manifests/featuregates/featureGate-Hypershift-TechPreviewNoUpgrade.yaml
@@ -15,6 +15,9 @@
         "featureGates": [
             {
                 "disabled": [
+                    {
+                        "name": "AdditionalRoutingCapabilities"
+                    },
                     {
                         "name": "ClusterAPIInstall"
                     },
@@ -44,9 +47,6 @@
                     {
                         "name": "AWSEFSDriverVolumeMetrics"
                     },
-                    {
-                        "name": "AdditionalRoutingCapabilities"
-                    },
                     {
                         "name": "AdminNetworkPolicy"
                     },
diff --git a/payload-manifests/featuregates/featureGate-SelfManagedHA-Default.yaml b/payload-manifests/featuregates/featureGate-SelfManagedHA-Default.yaml
@@ -16,9 +16,6 @@
                     {
                         "name": "AWSClusterHostedDNS"
                     },
-                    {
-                        "name": "AdditionalRoutingCapabilities"
-                    },
                     {
                         "name": "AutomatedEtcdBackup"
                     },
@@ -180,6 +177,9 @@
                     {
                         "name": "AWSEFSDriverVolumeMetrics"
                     },
+                    {
+                        "name": "AdditionalRoutingCapabilities"
+                    },
                     {
                         "name": "AdminNetworkPolicy"
                     },
diff --git a/payload-manifests/featuregates/featureGate-SelfManagedHA-DevPreviewNoUpgrade.yaml b/payload-manifests/featuregates/featureGate-SelfManagedHA-DevPreviewNoUpgrade.yaml
@@ -15,6 +15,9 @@
         "featureGates": [
             {
                 "disabled": [
+                    {
+                        "name": "AdditionalRoutingCapabilities"
+                    },
                     {
                         "name": "ClusterAPIInstall"
                     },
@@ -41,9 +44,6 @@
                     {
                         "name": "AWSEFSDriverVolumeMetrics"
                     },
-                    {
-                        "name": "AdditionalRoutingCapabilities"
-                    },
                     {
                         "name": "AdminNetworkPolicy"
                     },
diff --git a/payload-manifests/featuregates/featureGate-SelfManagedHA-TechPreviewNoUpgrade.yaml b/payload-manifests/featuregates/featureGate-SelfManagedHA-TechPreviewNoUpgrade.yaml
@@ -15,6 +15,9 @@
         "featureGates": [
             {
                 "disabled": [
+                    {
+                        "name": "AdditionalRoutingCapabilities"
+                    },
                     {
                         "name": "ClusterAPIInstall"
                     },
@@ -44,9 +47,6 @@
                     {
                         "name": "AWSEFSDriverVolumeMetrics"
                     },
-                    {
-                        "name": "AdditionalRoutingCapabilities"
-                    },
                     {
                         "name": "AdminNetworkPolicy"
                     },

Original file line number	Diff line number	Diff line change
`@@ -16,9 +16,6 @@`
`16`	`16`	`{`
`17`	`17`	`"name": "AWSClusterHostedDNS"`
`18`	`18`	`},`
`19`		`- {`
`20`		`- "name": "AdditionalRoutingCapabilities"`
`21`		`- },`
`22`	`19`	`{`
`23`	`20`	`"name": "AutomatedEtcdBackup"`
`24`	`21`	`},`
`@@ -177,6 +174,9 @@`
`177`	`174`	`{`
`178`	`175`	`"name": "AWSEFSDriverVolumeMetrics"`
`179`	`176`	`},`
	`177`	`+ {`
	`178`	`+ "name": "AdditionalRoutingCapabilities"`
	`179`	`+ },`
`180`	`180`	`{`
`181`	`181`	`"name": "AdminNetworkPolicy"`
`182`	`182`	`},`
Original file line number	Diff line number	Diff line change
`@@ -15,6 +15,9 @@`
`15`	`15`	`"featureGates": [`
`16`	`16`	`{`
`17`	`17`	`"disabled": [`
	`18`	`+ {`
	`19`	`+ "name": "AdditionalRoutingCapabilities"`
	`20`	`+ },`
`18`	`21`	`{`
`19`	`22`	`"name": "ClusterAPIInstall"`
`20`	`23`	`},`
`@@ -41,9 +44,6 @@`
`41`	`44`	`{`
`42`	`45`	`"name": "AWSEFSDriverVolumeMetrics"`
`43`	`46`	`},`
`44`		`- {`
`45`		`- "name": "AdditionalRoutingCapabilities"`
`46`		`- },`
`47`	`47`	`{`
`48`	`48`	`"name": "AdminNetworkPolicy"`
`49`	`49`	`},`