Add centralized logging for feature gate status

Signed-off-by: Brett Tofel <[email protected]>

Author: bentito

cmd/operator-controller/main.go

@@ -187,6 +187,9 @@ func run() error {
 
 	setupLog.Info("starting up the controller", "version info", version.String())
 
+	// log feature gate status after parsing flags and setting up logger
+	features.LogFeatureGateStates(setupLog, features.OperatorControllerFeatureGate)
+
 	authFilePath := filepath.Join(os.TempDir(), fmt.Sprintf("%s-%s.json", authFilePrefix, apimachineryrand.String(8)))
 	var globalPullSecretKey *k8stypes.NamespacedName
 	if cfg.globalPullSecret != "" {
@@ -419,10 +422,7 @@ func run() error {
 	// determine if PreAuthorizer should be enabled based on feature gate
 	var preAuth authorization.PreAuthorizer
 	if features.OperatorControllerFeatureGate.Enabled(features.PreflightPermissions) {
-		setupLog.Info("preflight permissions check enabled via feature gate")
 		preAuth = authorization.NewRBACPreAuthorizer(mgr.GetClient())
-	} else {
-		setupLog.Info("preflight permissions check disabled via feature gate")
 	}
 
 	// now initialize the helmApplier, assigning the potentially nil preAuth

internal/operator-controller/features/features.go

@@ -1,6 +1,9 @@
 package features
 
 import (
+	"sort"
+
+	"github.com/go-logr/logr"
 	utilruntime "k8s.io/apimachinery/pkg/util/runtime"
 	"k8s.io/component-base/featuregate"
 )
@@ -36,3 +39,20 @@ var OperatorControllerFeatureGate featuregate.MutableFeatureGate = featuregate.N
 func init() {
 	utilruntime.Must(OperatorControllerFeatureGate.Add(operatorControllerFeatureGates))
 }
+
+// LogFeatureGateStates logs the state of all known feature gates.
+func LogFeatureGateStates(log logr.Logger, fg featuregate.FeatureGate) {
+	// Sort the keys for consistent logging order
+	featureKeys := make([]featuregate.Feature, 0, len(operatorControllerFeatureGates))
+	for k := range operatorControllerFeatureGates {
+		featureKeys = append(featureKeys, k)
+	}
+	sort.Slice(featureKeys, func(i, j int) bool {
+		return string(featureKeys[i]) < string(featureKeys[j]) // Sort by string representation
+	})
+
+	log.Info("Feature Gates Status:")
+	for _, feature := range featureKeys {
+		log.Info("  ", "feature", string(feature), "enabled", fg.Enabled(feature))
+	}
+}