Use Structure Logging Enhancement for Cluster Resource Binding Controller (#6576)

* use structured logging for cluster resource binding controller

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

* Improve logs by using 'name' as key and log name only

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

---------

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

Author: zhuyulicfc49

pkg/controllers/status/crb_status_controller.go

@@ -58,7 +58,7 @@ type CRBStatusController struct {
 // The Controller will requeue the Request to be processed again if an error is non-nil or
 // Result.Requeue is true, otherwise upon completion it will remove the work from the queue.
 func (c *CRBStatusController) Reconcile(ctx context.Context, req controllerruntime.Request) (controllerruntime.Result, error) {
-	klog.V(4).Infof("Reconciling ClusterResourceBinding %s.", req.NamespacedName.String())
+	klog.V(4).InfoS("Reconciling ClusterResourceBinding", "name", req.NamespacedName.Name)
 
 	binding := &workv1alpha2.ClusterResourceBinding{}
 	if err := c.Client.Get(ctx, req.NamespacedName, binding); err != nil {
@@ -112,8 +112,7 @@ func (c *CRBStatusController) SetupWithManager(mgr controllerruntime.Manager) er
 func (c *CRBStatusController) syncBindingStatus(ctx context.Context, binding *workv1alpha2.ClusterResourceBinding) error {
 	err := helper.AggregateClusterResourceBindingWorkStatus(ctx, c.Client, binding, c.EventRecorder)
 	if err != nil {
-		klog.Errorf("Failed to aggregate workStatues to clusterResourceBinding(%s), Error: %v",
-			binding.Name, err)
+		klog.ErrorS(err, "Failed to aggregate workStatues to clusterResourceBinding", "name", binding.Name)
 		return err
 	}