reduce unnecessary watch events in the work-status-controller

Signed-off-by: changzhen <[email protected]>

Author: XiShanYongYe-Chang

cmd/agent/app/agent.go

@@ -366,6 +366,7 @@ func startWorkStatusController(ctx controllerscontext.Context) (bool, error) {
 		InformerManager:             genericmanager.GetInstance(),
 		Context:                     ctx.Context,
 		ObjectWatcher:               ctx.ObjectWatcher,
+		WorkPredicateFunc:           helper.NewWorkStatusPredicateOnAgent(ctx.Opts.ClusterName),
 		ClusterDynamicClientSetFunc: util.NewClusterDynamicClientSetForAgent,
 		ClusterCacheSyncTimeout:     ctx.Opts.ClusterCacheSyncTimeout,
 		ConcurrentWorkStatusSyncs:   ctx.Opts.ConcurrentWorkSyncs,

cmd/controller-manager/app/controllermanager.go

@@ -475,7 +475,7 @@ func startWorkStatusController(ctx controllerscontext.Context) (enabled bool, er
 		InformerManager:             genericmanager.GetInstance(),
 		Context:                     ctx.Context,
 		ObjectWatcher:               ctx.ObjectWatcher,
-		WorkPredicateFunc:           helper.WorkWithinPushClusterPredicate(ctx.Mgr),
+		WorkPredicateFunc:           helper.NewWorkStatusPredicate(ctx.Mgr),
 		ClusterDynamicClientSetFunc: util.NewClusterDynamicClientSet,
 		ClusterClientOption:         ctx.ClusterClientOption,
 		ClusterCacheSyncTimeout:     opts.ClusterCacheSyncTimeout,

pkg/controllers/status/work_status_controller.go

@@ -543,18 +543,13 @@ func (c *WorkStatusController) getSingleClusterManager(cluster *clusterv1alpha1.
 
 // SetupWithManager creates a controller and register to controller manager.
 func (c *WorkStatusController) SetupWithManager(mgr controllerruntime.Manager) error {
-	ctrlBuilder := controllerruntime.NewControllerManagedBy(mgr).Named(WorkStatusControllerName).
+	return controllerruntime.NewControllerManagedBy(mgr).
+		Named(WorkStatusControllerName).
+		For(&workv1alpha1.Work{}, builder.WithPredicates(c.WorkPredicateFunc)).
 		WithOptions(controller.Options{
 			RateLimiter: ratelimiterflag.DefaultControllerRateLimiter[controllerruntime.Request](c.RateLimiterOptions),
-		})
-
-	if c.WorkPredicateFunc != nil {
-		ctrlBuilder.For(&workv1alpha1.Work{}, builder.WithPredicates(c.WorkPredicateFunc))
-	} else {
-		ctrlBuilder.For(&workv1alpha1.Work{})
-	}
-
-	return ctrlBuilder.Complete(c)
+		}).
+		Complete(c)
 }
 
 func (c *WorkStatusController) eventf(object *unstructured.Unstructured, eventType, reason, messageFmt string, args ...interface{}) {

pkg/util/helper/predicate.go

@@ -29,10 +29,7 @@ import (
 	"github.com/karmada-io/karmada/pkg/util/names"
 )
 
-// WorkWithinPushClusterPredicate generates the event filter function to skip events that the controllers are uninterested.
-// Used by controllers:
-// - execution controller working in karmada-controller-manager
-// - work status controller working in karmada-controller-manager
+// WorkWithinPushClusterPredicate generates the event filter function for execution-controller in karmada-controller-manager
 func WorkWithinPushClusterPredicate(mgr controllerruntime.Manager) predicate.Funcs {
 	predFunc := func(object client.Object) bool {
 		obj := object.(*workv1alpha1.Work)
@@ -68,6 +65,81 @@ func WorkWithinPushClusterPredicate(mgr controllerruntime.Manager) predicate.Fun
 	}
 }
 
+// NewWorkStatusPredicate generates the event filter function for work-status-controller in karmada-controller-manager
+func NewWorkStatusPredicate(mgr controllerruntime.Manager) predicate.Funcs {
+	predFunc := func(object client.Object) bool {
+		obj := object.(*workv1alpha1.Work)
+
+		clusterName, err := names.GetClusterName(obj.Namespace)
+		if err != nil {
+			klog.Errorf("Failed to get member cluster name for work %s/%s", obj.Namespace, obj.Name)
+			return false
+		}
+
+		clusterObj, err := util.GetCluster(mgr.GetClient(), clusterName)
+		if err != nil {
+			klog.Errorf("Failed to get the given member cluster %s", clusterName)
+			return false
+		}
+
+		return clusterObj.Spec.SyncMode == clusterv1alpha1.Push
+	}
+	return predicate.Funcs{
+		CreateFunc: func(createEvent event.CreateEvent) bool {
+			return predFunc(createEvent.Object)
+		},
+		UpdateFunc: func(updateEvent event.UpdateEvent) bool {
+			workOld := updateEvent.ObjectOld.(*workv1alpha1.Work)
+			workNew := updateEvent.ObjectNew.(*workv1alpha1.Work)
+			workOldApplied := IsResourceApplied(&workOld.Status)
+			workNewApplied := IsResourceApplied(&workNew.Status)
+			if !workOldApplied && workNewApplied {
+				return predFunc(updateEvent.ObjectNew)
+			}
+			return false
+		},
+		DeleteFunc: func(event.DeleteEvent) bool {
+			return false
+		},
+		GenericFunc: func(event.GenericEvent) bool {
+			return false
+		},
+	}
+}
+
+// NewWorkStatusPredicateOnAgent generates the event filter function for work-status-controller in karmada-agent
+func NewWorkStatusPredicateOnAgent(curClusterName string) predicate.Funcs {
+	predFunc := func(object client.Object) bool {
+		clusterName, err := names.GetClusterName(object.GetNamespace())
+		if err != nil {
+			klog.Errorf("Failed to get member cluster name for work %s/%s", object.GetNamespace(), object.GetName())
+			return false
+		}
+		return clusterName == curClusterName
+	}
+	return predicate.Funcs{
+		CreateFunc: func(createEvent event.CreateEvent) bool {
+			return predFunc(createEvent.Object)
+		},
+		UpdateFunc: func(updateEvent event.UpdateEvent) bool {
+			workOld := updateEvent.ObjectOld.(*workv1alpha1.Work)
+			workNew := updateEvent.ObjectNew.(*workv1alpha1.Work)
+			workOldApplied := IsResourceApplied(&workOld.Status)
+			workNewApplied := IsResourceApplied(&workNew.Status)
+			if !workOldApplied && workNewApplied {
+				return predFunc(updateEvent.ObjectNew)
+			}
+			return false
+		},
+		DeleteFunc: func(event.DeleteEvent) bool {
+			return false
+		},
+		GenericFunc: func(event.GenericEvent) bool {
+			return false
+		},
+	}
+}
+
 // NewPredicateForServiceExportController generates an event filter function for ServiceExport controller running by karmada-controller-manager.
 func NewPredicateForServiceExportController(mgr controllerruntime.Manager) predicate.Funcs {
 	predFunc := func(eventType string, object client.Object) bool {