From 3b546603d84c6738ba4db3dea7439ccc545af5c3 Mon Sep 17 00:00:00 2001 From: Damiano Donati Date: Mon, 31 Aug 2026 10:40:30 +0200 Subject: [PATCH] OCPBUGS-105398: refactor: remove AzureWorkloadIdentity feature gate The gate is now GA and enabled by default upstream (openshift/api), so the unconditional featureGates.Enabled() call for it was always true. Once the gate is deregistered upstream, that call panics because library-go's FeatureGate.Enabled() panics for unknown feature names. Drop the map entry so the deployment no longer depends on the gate's registration status. --- pkg/operator/operator.go | 1 - pkg/operator/operator_test.go | 2 -- 2 files changed, 3 deletions(-) diff --git a/pkg/operator/operator.go b/pkg/operator/operator.go index 23f4715c6..fd6e172d6 100644 --- a/pkg/operator/operator.go +++ b/pkg/operator/operator.go @@ -477,7 +477,6 @@ func (optr *Operator) maoConfigFromInfrastructure() (*OperatorConfig, error) { features := map[string]bool{ string(apifeatures.FeatureGateAWSDedicatedHosts): featureGates.Enabled(apifeatures.FeatureGateAWSDedicatedHosts), string(apifeatures.FeatureGateMachineAPIMigration): featureGates.Enabled(apifeatures.FeatureGateMachineAPIMigration), - string(apifeatures.FeatureGateAzureWorkloadIdentity): featureGates.Enabled(apifeatures.FeatureGateAzureWorkloadIdentity), string(apifeatures.FeatureGateVSphereHostVMGroupZonal): featureGates.Enabled(apifeatures.FeatureGateVSphereHostVMGroupZonal), } if features[string(apifeatures.FeatureGateMachineAPIMigration)] { diff --git a/pkg/operator/operator_test.go b/pkg/operator/operator_test.go index 6a1cb43fe..6da986a1e 100644 --- a/pkg/operator/operator_test.go +++ b/pkg/operator/operator_test.go @@ -42,7 +42,6 @@ const ( var ( enabledFeatureGates = []openshiftv1.FeatureGateAttributes{ {Name: apifeatures.FeatureGateMachineAPIMigration}, - {Name: apifeatures.FeatureGateAzureWorkloadIdentity}, {Name: apifeatures.FeatureGateVSphereHostVMGroupZonal}, {Name: apifeatures.FeatureGateAWSDedicatedHosts}, } @@ -54,7 +53,6 @@ var ( enabledFeatureMap = map[string]bool{ "MachineAPIMigration": true, - "AzureWorkloadIdentity": true, "VSphereHostVMGroupZonal": true, "AWSDedicatedHosts": true, }