Remove the featuregate to enable DRA(DynamicResourceAllocation) by default

sairameshv · sairameshv · commit fad9674e9bd2 · 2025-09-24T16:38:00.000+05:30
DRA has been graduated to GA in upstream 1.34 It makes sense to remove the featuregate related to the DRA in the downstream This is also required to unblok the e2e test failures encountered during the OCP-4.21 rebase Reference for the failure: https://issues.redhat.com/browse/OCPBUGS-61381 Signed-off-by: Sai Ramesh Vanka <svanka@redhat.com>
diff --git a/config/v1/zz_generated.crd-manifests/0000_10_config-operator_01_schedulers-DevPreviewNoUpgrade.crd.yaml b/config/v1/zz_generated.crd-manifests/0000_10_config-operator_01_schedulers-DevPreviewNoUpgrade.crd.yaml
@@ -109,26 +109,6 @@ spec:
                 - HighNodeUtilization
                 - NoScoring
                 type: string
-              profileCustomizations:
-                description: profileCustomizations contains configuration for modifying
-                  the default behavior of existing scheduler profiles.
-                properties:
-                  dynamicResourceAllocation:
-                    description: |-
-                      dynamicResourceAllocation allows to enable or disable dynamic resource allocation within the scheduler.
-                      Dynamic resource allocation is an API for requesting and sharing resources between pods and containers inside a pod.
-                      Third-party resource drivers are responsible for tracking and allocating resources.
-                      Different kinds of resources support arbitrary parameters for defining requirements and initialization.
-                      Valid values are Enabled, Disabled and omitted.
-                      When omitted, this means no opinion and the platform is left to choose a reasonable default,
-                      which is subject to change over time.
-                      The current default is Disabled.
-                    enum:
-                    - ""
-                    - Enabled
-                    - Disabled
-                    type: string
-                type: object
             type: object
           status:
             description: status holds observed values from the cluster. They may not
diff --git a/config/v1/zz_generated.crd-manifests/0000_10_config-operator_01_schedulers-TechPreviewNoUpgrade.crd.yaml b/config/v1/zz_generated.crd-manifests/0000_10_config-operator_01_schedulers-TechPreviewNoUpgrade.crd.yaml
@@ -109,26 +109,6 @@ spec:
                 - HighNodeUtilization
                 - NoScoring
                 type: string
-              profileCustomizations:
-                description: profileCustomizations contains configuration for modifying
-                  the default behavior of existing scheduler profiles.
-                properties:
-                  dynamicResourceAllocation:
-                    description: |-
-                      dynamicResourceAllocation allows to enable or disable dynamic resource allocation within the scheduler.
-                      Dynamic resource allocation is an API for requesting and sharing resources between pods and containers inside a pod.
-                      Third-party resource drivers are responsible for tracking and allocating resources.
-                      Different kinds of resources support arbitrary parameters for defining requirements and initialization.
-                      Valid values are Enabled, Disabled and omitted.
-                      When omitted, this means no opinion and the platform is left to choose a reasonable default,
-                      which is subject to change over time.
-                      The current default is Disabled.
-                    enum:
-                    - ""
-                    - Enabled
-                    - Disabled
-                    type: string
-                type: object
             type: object
           status:
             description: status holds observed values from the cluster. They may not
diff --git a/features.md b/features.md
@@ -36,7 +36,6 @@
 | DNSNameResolver| | | <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>  |
 | DualReplica| | | <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>  |
 | DyanmicServiceEndpointIBMCloud| | | <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>  |
-| DynamicResourceAllocation| | | <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>  |
 | EtcdBackendQuota| | | <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>  |
 | Example| | | <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>  |
 | ExternalOIDCWithUIDAndExtraClaimMappings| | | <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
@@ -102,14 +102,6 @@ var (
 				enableIn(configv1.DevPreviewNoUpgrade, configv1.TechPreviewNoUpgrade).
 				mustRegister()
 
-	FeatureGateDynamicResourceAllocation = newFeatureGate("DynamicResourceAllocation").
-						reportProblemsToJiraComponent("scheduling").
-						contactPerson("jchaloup").
-						productScope(kubernetes).
-						enhancementPR("https://github.com/kubernetes/enhancements/issues/4381").
-						enableIn(configv1.DevPreviewNoUpgrade, configv1.TechPreviewNoUpgrade).
-						mustRegister()
-
 	FeatureGateAzureWorkloadIdentity = newFeatureGate("AzureWorkloadIdentity").
 						reportProblemsToJiraComponent("cloud-credential-operator").
 						contactPerson("abutcher").
diff --git a/payload-manifests/crds/0000_10_config-operator_01_schedulers-DevPreviewNoUpgrade.crd.yaml b/payload-manifests/crds/0000_10_config-operator_01_schedulers-DevPreviewNoUpgrade.crd.yaml
@@ -109,26 +109,6 @@ spec:
                 - HighNodeUtilization
                 - NoScoring
                 type: string
-              profileCustomizations:
-                description: profileCustomizations contains configuration for modifying
-                  the default behavior of existing scheduler profiles.
-                properties:
-                  dynamicResourceAllocation:
-                    description: |-
-                      dynamicResourceAllocation allows to enable or disable dynamic resource allocation within the scheduler.
-                      Dynamic resource allocation is an API for requesting and sharing resources between pods and containers inside a pod.
-                      Third-party resource drivers are responsible for tracking and allocating resources.
-                      Different kinds of resources support arbitrary parameters for defining requirements and initialization.
-                      Valid values are Enabled, Disabled and omitted.
-                      When omitted, this means no opinion and the platform is left to choose a reasonable default,
-                      which is subject to change over time.
-                      The current default is Disabled.
-                    enum:
-                    - ""
-                    - Enabled
-                    - Disabled
-                    type: string
-                type: object
             type: object
           status:
             description: status holds observed values from the cluster. They may not
diff --git a/payload-manifests/crds/0000_10_config-operator_01_schedulers-TechPreviewNoUpgrade.crd.yaml b/payload-manifests/crds/0000_10_config-operator_01_schedulers-TechPreviewNoUpgrade.crd.yaml
@@ -109,26 +109,6 @@ spec:
                 - HighNodeUtilization
                 - NoScoring
                 type: string
-              profileCustomizations:
-                description: profileCustomizations contains configuration for modifying
-                  the default behavior of existing scheduler profiles.
-                properties:
-                  dynamicResourceAllocation:
-                    description: |-
-                      dynamicResourceAllocation allows to enable or disable dynamic resource allocation within the scheduler.
-                      Dynamic resource allocation is an API for requesting and sharing resources between pods and containers inside a pod.
-                      Third-party resource drivers are responsible for tracking and allocating resources.
-                      Different kinds of resources support arbitrary parameters for defining requirements and initialization.
-                      Valid values are Enabled, Disabled and omitted.
-                      When omitted, this means no opinion and the platform is left to choose a reasonable default,
-                      which is subject to change over time.
-                      The current default is Disabled.
-                    enum:
-                    - ""
-                    - Enabled
-                    - Disabled
-                    type: string
-                type: object
             type: object
           status:
             description: status holds observed values from the cluster. They may not
diff --git a/payload-manifests/featuregates/featureGate-Hypershift-Default.yaml b/payload-manifests/featuregates/featureGate-Hypershift-Default.yaml
@@ -82,9 +82,6 @@
                     {
                         "name": "DyanmicServiceEndpointIBMCloud"
                     },
-                    {
-                        "name": "DynamicResourceAllocation"
-                    },
                     {
                         "name": "EtcdBackendQuota"
                     },
diff --git a/payload-manifests/featuregates/featureGate-Hypershift-DevPreviewNoUpgrade.yaml b/payload-manifests/featuregates/featureGate-Hypershift-DevPreviewNoUpgrade.yaml
@@ -137,9 +137,6 @@
                     {
                         "name": "DyanmicServiceEndpointIBMCloud"
                     },
-                    {
-                        "name": "DynamicResourceAllocation"
-                    },
                     {
                         "name": "EtcdBackendQuota"
                     },
diff --git a/payload-manifests/featuregates/featureGate-Hypershift-TechPreviewNoUpgrade.yaml b/payload-manifests/featuregates/featureGate-Hypershift-TechPreviewNoUpgrade.yaml
@@ -146,9 +146,6 @@
                     {
                         "name": "DyanmicServiceEndpointIBMCloud"
                     },
-                    {
-                        "name": "DynamicResourceAllocation"
-                    },
                     {
                         "name": "EtcdBackendQuota"
                     },
diff --git a/payload-manifests/featuregates/featureGate-SelfManagedHA-Default.yaml b/payload-manifests/featuregates/featureGate-SelfManagedHA-Default.yaml
@@ -82,9 +82,6 @@
                     {
                         "name": "DyanmicServiceEndpointIBMCloud"
                     },
-                    {
-                        "name": "DynamicResourceAllocation"
-                    },
                     {
                         "name": "EtcdBackendQuota"
                     },
diff --git a/payload-manifests/featuregates/featureGate-SelfManagedHA-DevPreviewNoUpgrade.yaml b/payload-manifests/featuregates/featureGate-SelfManagedHA-DevPreviewNoUpgrade.yaml
@@ -119,9 +119,6 @@
                     {
                         "name": "DyanmicServiceEndpointIBMCloud"
                     },
-                    {
-                        "name": "DynamicResourceAllocation"
-                    },
                     {
                         "name": "EtcdBackendQuota"
                     },
diff --git a/payload-manifests/featuregates/featureGate-SelfManagedHA-TechPreviewNoUpgrade.yaml b/payload-manifests/featuregates/featureGate-SelfManagedHA-TechPreviewNoUpgrade.yaml
@@ -128,9 +128,6 @@
                     {
                         "name": "DyanmicServiceEndpointIBMCloud"
                     },
-                    {
-                        "name": "DynamicResourceAllocation"
-                    },
                     {
                         "name": "EtcdBackendQuota"
                     },

Original file line number	Diff line number	Diff line change
`@@ -82,9 +82,6 @@`
`82`	`82`	`{`
`83`	`83`	`"name": "DyanmicServiceEndpointIBMCloud"`
`84`	`84`	`},`
`85`		`- {`
`86`		`- "name": "DynamicResourceAllocation"`
`87`		`- },`
`88`	`85`	`{`
`89`	`86`	`"name": "EtcdBackendQuota"`
`90`	`87`	`},`
Original file line number	Diff line number	Diff line change
`@@ -137,9 +137,6 @@`
`137`	`137`	`{`
`138`	`138`	`"name": "DyanmicServiceEndpointIBMCloud"`
`139`	`139`	`},`
`140`		`- {`
`141`		`- "name": "DynamicResourceAllocation"`
`142`		`- },`
`143`	`140`	`{`
`144`	`141`	`"name": "EtcdBackendQuota"`
`145`	`142`	`},`
Original file line number	Diff line number	Diff line change
`@@ -146,9 +146,6 @@`
`146`	`146`	`{`
`147`	`147`	`"name": "DyanmicServiceEndpointIBMCloud"`
`148`	`148`	`},`
`149`		`- {`
`150`		`- "name": "DynamicResourceAllocation"`
`151`		`- },`
`152`	`149`	`{`
`153`	`150`	`"name": "EtcdBackendQuota"`
`154`	`151`	`},`