MON-3971: Remove MetricsServer feature gate

slashpai · slashpai · commit 49ba8ebd1118 · 2024-09-02T12:18:25.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>  |
 | MultiArchInstallAWS| <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>  |
 | MultiArchInstallGCP| <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>  |
diff --git a/features/features.go b/features/features.go
@@ -292,13 +292,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-Default.yaml b/payload-manifests/featuregates/featureGate-Hypershift-Default.yaml
@@ -225,9 +225,6 @@
                     {
                         "name": "ManagedBootImages"
                     },
-                    {
-                        "name": "MetricsServer"
-                    },
                     {
                         "name": "MultiArchInstallAWS"
                     },
diff --git a/payload-manifests/featuregates/featureGate-Hypershift-DevPreviewNoUpgrade.yaml b/payload-manifests/featuregates/featureGate-Hypershift-DevPreviewNoUpgrade.yaml
@@ -164,9 +164,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
@@ -164,9 +164,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": "MultiArchInstallAWS"
                     },
diff --git a/payload-manifests/featuregates/featureGate-SelfManagedHA-DevPreviewNoUpgrade.yaml b/payload-manifests/featuregates/featureGate-SelfManagedHA-DevPreviewNoUpgrade.yaml
@@ -164,9 +164,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
@@ -164,9 +164,6 @@
                     {
                         "name": "MetricsCollectionProfiles"
                     },
-                    {
-                        "name": "MetricsServer"
-                    },
                     {
                         "name": "MixedCPUsAllocation"
                     },

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": "MultiArchInstallAWS"`
`233`	`230`	`},`
Original file line number	Diff line number	Diff line change
`@@ -164,9 +164,6 @@`
`164`	`164`	`{`
`165`	`165`	`"name": "MetricsCollectionProfiles"`
`166`	`166`	`},`
`167`		`- {`
`168`		`- "name": "MetricsServer"`
`169`		`- },`
`170`	`167`	`{`
`171`	`168`	`"name": "MixedCPUsAllocation"`
`172`	`169`	`},`