MON-3971: Remove MetricsServer feature gate

slashpai · slashpai · commit 7b2ea98ad489 · 2024-08-07T13:09:10.000+05:30
cluster-monitoring-operator now by default installs
and configures metrics-server and it doesn't need
to be behind feature gate anymore.

Signed-off-by: Jayapriya Pai &lt;janantha@redhat.com&gt;
diff --git a/features.md b/features.md
@@ -70,7 +70,6 @@
 | IngressControllerLBSubnetsAWS| <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> | <span style="background-color: #519450">Enabled</span> | <span style="background-color: #519450">Enabled</span>  |
 | KMSv1| <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> | <span style="background-color: #519450">Enabled</span> | <span style="background-color: #519450">Enabled</span>  |
 | ManagedBootImages| <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> | <span style="background-color: #519450">Enabled</span> | <span style="background-color: #519450">Enabled</span>  |
-| MetricsServer| <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> | <span style="background-color: #519450">Enabled</span> | <span style="background-color: #519450">Enabled</span>  |
 | NetworkDiagnosticsConfig| <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> | <span style="background-color: #519450">Enabled</span> | <span style="background-color: #519450">Enabled</span>  |
 | NetworkLiveMigration| <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> | <span style="background-color: #519450">Enabled</span> | <span style="background-color: #519450">Enabled</span>  |
 | NodeDisruptionPolicy| <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> | <span style="background-color: #519450">Enabled</span> | <span style="background-color: #519450">Enabled</span>  |
diff --git a/features/features.go b/features/features.go
@@ -306,13 +306,6 @@ var (
 					productScope(ocpSpecific).
 					mustRegister()
 
-	FeatureGateMetricsServer = newFeatureGate("MetricsServer").
-					reportProblemsToJiraComponent("Monitoring").
-					contactPerson("slashpai").
-					productScope(ocpSpecific).
-					enableIn(configv1.Default, configv1.DevPreviewNoUpgrade, configv1.TechPreviewNoUpgrade).
-					mustRegister()
-
 	FeatureGateInstallAlternateInfrastructureAWS = newFeatureGate("InstallAlternateInfrastructureAWS").
 							reportProblemsToJiraComponent("Installer").
 							contactPerson("padillon").
diff --git a/payload-manifests/featuregates/featureGate-Hypershift-DevPreviewNoUpgrade.yaml b/payload-manifests/featuregates/featureGate-Hypershift-DevPreviewNoUpgrade.yaml
@@ -170,9 +170,6 @@
                     {
                         "name": "MetricsCollectionProfiles"
                     },
-                    {
-                        "name": "MetricsServer"
-                    },
                     {
                         "name": "MixedCPUsAllocation"
                     },
diff --git a/payload-manifests/featuregates/featureGate-Hypershift-TechPreviewNoUpgrade.yaml b/payload-manifests/featuregates/featureGate-Hypershift-TechPreviewNoUpgrade.yaml
@@ -170,9 +170,6 @@
                     {
                         "name": "MetricsCollectionProfiles"
                     },
-                    {
-                        "name": "MetricsServer"
-                    },
                     {
                         "name": "MixedCPUsAllocation"
                     },
diff --git a/payload-manifests/featuregates/featureGate-SelfManagedHA-Default.yaml b/payload-manifests/featuregates/featureGate-SelfManagedHA-Default.yaml
@@ -225,9 +225,6 @@
                     {
                         "name": "ManagedBootImages"
                     },
-                    {
-                        "name": "MetricsServer"
-                    },
                     {
                         "name": "NetworkDiagnosticsConfig"
                     },
diff --git a/payload-manifests/featuregates/featureGate-SelfManagedHA-DevPreviewNoUpgrade.yaml b/payload-manifests/featuregates/featureGate-SelfManagedHA-DevPreviewNoUpgrade.yaml
@@ -170,9 +170,6 @@
                     {
                         "name": "MetricsCollectionProfiles"
                     },
-                    {
-                        "name": "MetricsServer"
-                    },
                     {
                         "name": "MixedCPUsAllocation"
                     },
diff --git a/payload-manifests/featuregates/featureGate-SelfManagedHA-TechPreviewNoUpgrade.yaml b/payload-manifests/featuregates/featureGate-SelfManagedHA-TechPreviewNoUpgrade.yaml
@@ -170,9 +170,6 @@
                     {
                         "name": "MetricsCollectionProfiles"
                     },
-                    {
-                        "name": "MetricsServer"
-                    },
                     {
                         "name": "MixedCPUsAllocation"
                     },

Original file line number	Diff line number	Diff line change
`@@ -170,9 +170,6 @@`
`170`	`170`	`{`
`171`	`171`	`"name": "MetricsCollectionProfiles"`
`172`	`172`	`},`
`173`		`- {`
`174`		`- "name": "MetricsServer"`
`175`		`- },`
`176`	`173`	`{`
`177`	`174`	`"name": "MixedCPUsAllocation"`
`178`	`175`	`},`
Original file line number	Diff line number	Diff line change
`@@ -225,9 +225,6 @@`
`225`	`225`	`{`
`226`	`226`	`"name": "ManagedBootImages"`
`227`	`227`	`},`
`228`		`- {`
`229`		`- "name": "MetricsServer"`
`230`		`- },`
`231`	`228`	`{`
`232`	`229`	`"name": "NetworkDiagnosticsConfig"`
`233`	`230`	`},`