Do not append 'namespaceAll' to managed namespaces on initialization.

elastic · Dec 13, 2021 · 62804ac · 62804ac
1 parent 9ddb3cf
commit 62804ac
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/cmd/manager/main.go b/cmd/manager/main.go
@@ -479,12 +479,12 @@ func startOperator(ctx context.Context) error {
 		// The managed cache should always include the operator namespace so that we can work with operator-internal resources.
 		managedNamespaces = append(managedNamespaces, operatorNamespace)
 
+		opts.NewCache = cache.MultiNamespacedCacheBuilder(managedNamespaces)
+
 		// Add the empty namespace to allow watching cluster-scoped resources if storage class validation is enabled.
 		if viper.GetBool(operator.ValidateStorageClassFlag) {
-			managedNamespaces = append(managedNamespaces, "")
+			opts.NewCache = cache.MultiNamespacedCacheBuilder(append(managedNamespaces, ""))
 		}
-
-		opts.NewCache = cache.MultiNamespacedCacheBuilder(managedNamespaces)
 	}
 
 	// only expose prometheus metrics if provided a non-zero port