feat: Changed to V(4) log

Author: Abrahami2

populator-machinery/controller.go

@@ -613,7 +613,6 @@ func (c *controller) runWorker() {
 }
 
 func (c *controller) syncPvc(ctx context.Context, key, pvcNamespace, pvcName string) error {
-	klog.V(1).Infof("Starting sync for PVC %s/%s", pvcNamespace, pvcName)
 	if c.populatorNamespace == pvcNamespace {
 		// Ignore PVCs in our own working namespace
 		return nil
@@ -689,15 +688,15 @@ func (c *controller) syncPvc(ctx context.Context, key, pvcNamespace, pvcName str
 		}
 
 		if err := c.checkIntreeStorageClass(pvc, storageClass); err != nil {
-			klog.V(2).Infof("Ignoring PVC %s/%s: %s", pvcNamespace, pvcName, err)
+			klog.V(4).Infof("Ignoring PVC %s/%s: %s", pvcNamespace, pvcName, err)
 			return nil
 		}
 
 		if storageClass.VolumeBindingMode != nil && storagev1.VolumeBindingWaitForFirstConsumer == *storageClass.VolumeBindingMode {
 			waitForFirstConsumer = true
 			nodeName = pvc.Annotations[annSelectedNode]
 			if nodeName == "" {
-				klog.V(2).InfoS("Waiting for node selection due to WaitForFirstConsumer mode", "pvc", klog.KObj(pvc))
+				klog.V(4).Infof("Waiting for node selection due to WaitForFirstConsumer mode for PVC %s/%s", pvc.Namespace, pvc.Name)
 				// Wait for the PVC to get a node name before continuing
 				return nil
 			}
@@ -783,7 +782,7 @@ func (c *controller) syncPvc(ctx context.Context, key, pvcNamespace, pvcName str
 				return err
 			}
 			params.PvcPrime = pvcPrime
-			klog.V(2).InfoS("Created PVC Prime for data population", "pvc", klog.KObj(pvc), "pvcPrime", klog.KObj(pvcPrime))
+			klog.V(4).Infof("Created pvcPrime %s/%s for data population of PVC %s/%s", pvcPrime.Namespace, pvcPrime.Name, pvc.Namespace, pvc.Name)
 		}
 
 		if "" == pvc.Spec.VolumeName {
@@ -803,7 +802,6 @@ func (c *controller) syncPvc(ctx context.Context, key, pvcNamespace, pvcName str
 				if c.providerFunctionConfig.PopulateFn != nil {
 
 					if "" == pvcPrime.Spec.VolumeName {
-						klog.V(2).InfoS("Waiting for PVC Prime to be bound", "pvc", klog.KObj(pvc), "pvcPrime", klog.KObj(pvcPrime))
 						// We'll get called again later when the pvcPrime gets bounded
 						return nil
 					}
@@ -822,7 +820,7 @@ func (c *controller) syncPvc(ctx context.Context, key, pvcNamespace, pvcName str
 						return err
 					}
 					if !complete {
-						klog.Infof("Data population for PVC %s/%s is not yet complete. Requeuing to check again later.", pvc.Namespace, pvc.Name)
+						klog.V(4).Infof("Data population for PVC %s/%s is not yet complete. Requeuing to check again later.", pvc.Namespace, pvc.Name)
 						// TODO: Revisited if there is a better way to requeue pvc than return an error
 						// Return error to force reque pvc. We'll get called again later when the population operation complete
 						return fmt.Errorf(reasonWaitForDataPopulationFinished)
@@ -886,7 +884,7 @@ func (c *controller) syncPvc(ctx context.Context, key, pvcNamespace, pvcName str
 						return err
 					}
 					c.recorder.Eventf(pvc, corev1.EventTypeNormal, reasonPodCreationSuccess, "Populator started")
-					klog.V(2).InfoS("Created populator pod", "pvc", klog.KObj(pvc), "pod", klog.KObj(pod))
+					klog.V(4).Infof("Created populator pod %s/%s for PVC %s/%s", pod.Namespace, pod.Name, pvc.Namespace, pvc.Name)
 
 					// We'll get called again later when the pod exists
 					return nil
@@ -901,7 +899,7 @@ func (c *controller) syncPvc(ctx context.Context, key, pvcNamespace, pvcName str
 						if err != nil {
 							return err
 						}
-						klog.V(2).InfoS("Cleaned up populator pod", "pvc", klog.KObj(pvc), "pod", klog.KObj(pod))
+						klog.V(4).Infof("Cleaned up temporary populator pod %s/%s for PVC %s/%s", pod.Namespace, pod.Name, pvc.Namespace, pvc.Name)
 					}
 					// We'll get called again later when the pod succeeds
 					return nil
@@ -954,7 +952,7 @@ func (c *controller) syncPvc(ctx context.Context, key, pvcNamespace, pvcName str
 				if err != nil {
 					return err
 				}
-				klog.V(2).InfoS("Successfully rebound PV to target PVC", "pv", klog.KObj(pv), "pvc", klog.KObj(pvc))
+				klog.V(4).Infof("Successfully rebound PV %s to target PVC %s/%s", pv.Name, pvc.Namespace, pvc.Name)
 
 				// Don't start cleaning up yet -- we need to bind controller to acknowledge
 				// the switch
@@ -1008,7 +1006,7 @@ func (c *controller) syncPvc(ctx context.Context, key, pvcNamespace, pvcName str
 	if err != nil {
 		return err
 	}
-	klog.V(2).InfoS("Cleaned up temporary populator resources", "pvc", klog.KObj(pvc))
+	klog.V(4).Infof("Cleaned up temporary populator resources for PVC %s/%s", pvc.Namespace, pvc.Name)
 
 	// Clean up our internal callback maps
 	c.cleanupNotifications(key)