Merge pull request #2242 from tmshort/preflight-checks

openshift-merge-bot[bot] · web-flow · commit 6fb534fddf16 · 2025-04-08T18:09:27.000Z
OPRUN-3780: Add feature flag for NewOLMPreflightPermissionCheck
diff --git a/features.md b/features.md
@@ -8,6 +8,7 @@
 | ClusterVersionOperatorConfiguration| | | <span style="background-color: #519450">Enabled</span> | <span style="background-color: #519450">Enabled</span> | |  |
 | Example2| | | <span style="background-color: #519450">Enabled</span> | <span style="background-color: #519450">Enabled</span> | |  |
 | NewOLMCatalogdAPIV1Metas| | | | <span style="background-color: #519450">Enabled</span> | | <span style="background-color: #519450">Enabled</span>  |
+| NewOLMPreflightPermissionChecks| | | | <span style="background-color: #519450">Enabled</span> | | <span style="background-color: #519450">Enabled</span>  |
 | SELinuxChangePolicy| | | <span style="background-color: #519450">Enabled</span> | <span style="background-color: #519450">Enabled</span> | |  |
 | SELinuxMount| | | <span style="background-color: #519450">Enabled</span> | <span style="background-color: #519450">Enabled</span> | |  |
 | ShortCertRotation| | | <span style="background-color: #519450">Enabled</span> | <span style="background-color: #519450">Enabled</span> | |  |
diff --git a/features/features.go b/features/features.go
@@ -530,6 +530,14 @@ var (
 						enableForClusterProfile(SelfManaged, configv1.DevPreviewNoUpgrade, configv1.TechPreviewNoUpgrade).
 						mustRegister()
 
+	FeatureGateNewOLMPreflightPermissionChecks = newFeatureGate("NewOLMPreflightPermissionChecks").
+							reportProblemsToJiraComponent("olm").
+							contactPerson("tshort").
+							productScope(ocpSpecific).
+							enhancementPR("https://github.com/openshift/enhancements/pull/1768").
+							enableForClusterProfile(SelfManaged, configv1.DevPreviewNoUpgrade, configv1.TechPreviewNoUpgrade).
+							mustRegister()
+
 	FeatureGateInsightsOnDemandDataGather = newFeatureGate("InsightsOnDemandDataGather").
 						reportProblemsToJiraComponent("insights").
 						contactPerson("tremes").
diff --git a/payload-manifests/featuregates/featureGate-Hypershift-Default.yaml b/payload-manifests/featuregates/featureGate-Hypershift-Default.yaml
@@ -133,6 +133,9 @@
                     {
                         "name": "NewOLMCatalogdAPIV1Metas"
                     },
+                    {
+                        "name": "NewOLMPreflightPermissionChecks"
+                    },
                     {
                         "name": "NodeSwap"
                     },
diff --git a/payload-manifests/featuregates/featureGate-Hypershift-DevPreviewNoUpgrade.yaml b/payload-manifests/featuregates/featureGate-Hypershift-DevPreviewNoUpgrade.yaml
@@ -35,6 +35,9 @@
                     },
                     {
                         "name": "NewOLMCatalogdAPIV1Metas"
+                    },
+                    {
+                        "name": "NewOLMPreflightPermissionChecks"
                     }
                 ],
                 "enabled": [
diff --git a/payload-manifests/featuregates/featureGate-Hypershift-TechPreviewNoUpgrade.yaml b/payload-manifests/featuregates/featureGate-Hypershift-TechPreviewNoUpgrade.yaml
@@ -42,6 +42,9 @@
                     {
                         "name": "NewOLMCatalogdAPIV1Metas"
                     },
+                    {
+                        "name": "NewOLMPreflightPermissionChecks"
+                    },
                     {
                         "name": "SELinuxChangePolicy"
                     },
diff --git a/payload-manifests/featuregates/featureGate-SelfManagedHA-Default.yaml b/payload-manifests/featuregates/featureGate-SelfManagedHA-Default.yaml
@@ -133,6 +133,9 @@
                     {
                         "name": "NewOLMCatalogdAPIV1Metas"
                     },
+                    {
+                        "name": "NewOLMPreflightPermissionChecks"
+                    },
                     {
                         "name": "NodeSwap"
                     },
diff --git a/payload-manifests/featuregates/featureGate-SelfManagedHA-DevPreviewNoUpgrade.yaml b/payload-manifests/featuregates/featureGate-SelfManagedHA-DevPreviewNoUpgrade.yaml
@@ -203,6 +203,9 @@
                     {
                         "name": "NewOLMCatalogdAPIV1Metas"
                     },
+                    {
+                        "name": "NewOLMPreflightPermissionChecks"
+                    },
                     {
                         "name": "NodeDisruptionPolicy"
                     },
diff --git a/payload-manifests/featuregates/featureGate-SelfManagedHA-TechPreviewNoUpgrade.yaml b/payload-manifests/featuregates/featureGate-SelfManagedHA-TechPreviewNoUpgrade.yaml
@@ -215,6 +215,9 @@
                     {
                         "name": "NewOLMCatalogdAPIV1Metas"
                     },
+                    {
+                        "name": "NewOLMPreflightPermissionChecks"
+                    },
                     {
                         "name": "NodeDisruptionPolicy"
                     },

Original file line number	Diff line number	Diff line change
`@@ -133,6 +133,9 @@`
`133`	`133`	`{`
`134`	`134`	`"name": "NewOLMCatalogdAPIV1Metas"`
`135`	`135`	`},`
	`136`	`+ {`
	`137`	`+ "name": "NewOLMPreflightPermissionChecks"`
	`138`	`+ },`
`136`	`139`	`{`
`137`	`140`	`"name": "NodeSwap"`
`138`	`141`	`},`
Original file line number	Diff line number	Diff line change
`@@ -35,6 +35,9 @@`
`35`	`35`	`},`
`36`	`36`	`{`
`37`	`37`	`"name": "NewOLMCatalogdAPIV1Metas"`
	`38`	`+ },`
	`39`	`+ {`
	`40`	`+ "name": "NewOLMPreflightPermissionChecks"`
`38`	`41`	`}`
`39`	`42`	`],`
`40`	`43`	`"enabled": [`
Original file line number	Diff line number	Diff line change
`@@ -42,6 +42,9 @@`
`42`	`42`	`{`
`43`	`43`	`"name": "NewOLMCatalogdAPIV1Metas"`
`44`	`44`	`},`
	`45`	`+ {`
	`46`	`+ "name": "NewOLMPreflightPermissionChecks"`
	`47`	`+ },`
`45`	`48`	`{`
`46`	`49`	`"name": "SELinuxChangePolicy"`
`47`	`50`	`},`
Original file line number	Diff line number	Diff line change
`@@ -203,6 +203,9 @@`
`203`	`203`	`{`
`204`	`204`	`"name": "NewOLMCatalogdAPIV1Metas"`
`205`	`205`	`},`
	`206`	`+ {`
	`207`	`+ "name": "NewOLMPreflightPermissionChecks"`
	`208`	`+ },`
`206`	`209`	`{`
`207`	`210`	`"name": "NodeDisruptionPolicy"`
`208`	`211`	`},`
Original file line number	Diff line number	Diff line change
`@@ -215,6 +215,9 @@`
`215`	`215`	`{`
`216`	`216`	`"name": "NewOLMCatalogdAPIV1Metas"`
`217`	`217`	`},`
	`218`	`+ {`
	`219`	`+ "name": "NewOLMPreflightPermissionChecks"`
	`220`	`+ },`
`218`	`221`	`{`
`219`	`222`	`"name": "NodeDisruptionPolicy"`
`220`	`223`	`},`