numaresources:controller: do not watch mcp

On HyperShift platform the MCP crds are not exist. Hence, we should setup a watches for them only when running on OCP platform. Signed-off-by: Talor Itzhak <[email protected]>
openshift-kni · Nov 18, 2024 · 719c5d5 · 719c5d5
1 parent 69bca7b
commit 719c5d5
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/controllers/numaresourcesoperator_controller.go b/controllers/numaresourcesoperator_controller.go
@@ -560,18 +560,18 @@ func (r *NUMAResourcesOperatorReconciler) SetupWithManager(mgr ctrl.Manager) err
 
 	b := ctrl.NewControllerManagedBy(mgr).For(&nropv1.NUMAResourcesOperator{})
 	if r.Platform == platform.OpenShift {
-		b = b.Owns(&securityv1.SecurityContextConstraints{}).
+		b.Watches(
+			&machineconfigv1.MachineConfigPool{},
+			handler.EnqueueRequestsFromMapFunc(r.mcpToNUMAResourceOperator),
+			builder.WithPredicates(mcpPredicates)).
+			Owns(&securityv1.SecurityContextConstraints{}).
 			Owns(&machineconfigv1.MachineConfig{}, builder.WithPredicates(p))
 	}
 	return b.Owns(&apiextensionv1.CustomResourceDefinition{}).
 		Owns(&corev1.ServiceAccount{}, builder.WithPredicates(p)).
 		Owns(&rbacv1.RoleBinding{}, builder.WithPredicates(p)).
 		Owns(&rbacv1.Role{}, builder.WithPredicates(p)).
 		Owns(&appsv1.DaemonSet{}, builder.WithPredicates(p)).
-		Watches(
-			&machineconfigv1.MachineConfigPool{},
-			handler.EnqueueRequestsFromMapFunc(r.mcpToNUMAResourceOperator),
-			builder.WithPredicates(mcpPredicates)).
 		Complete(r)
 }