Config: Move info logs to V(1)

Signed-off-by: Andreas Karis <[email protected]>

Author: andreaskaris

controllers/bpfman-operator/config.go

@@ -90,7 +90,7 @@ func (r *BpfmanConfigReconciler) SetupWithManager(mgr ctrl.Manager) error {
 
 func (r *BpfmanConfigReconciler) Reconcile(ctx context.Context, req ctrl.Request) (ctrl.Result, error) {
 	r.Logger = ctrl.Log.WithName("Config")
-	r.Logger.Info("Running the reconciler")
+	r.Logger.V(1).Info("Running the reconciler", "req.NamespacedName", req.NamespacedName)
 
 	bpfmanConfig := &v1alpha1.Config{}
 	// We must use types.NamespacedName{Name: req.NamespacedName.Name} instead of req.NamespacedName. Otherwise, the
@@ -114,7 +114,7 @@ func (r *BpfmanConfigReconciler) Reconcile(ctx context.Context, req ctrl.Request
 	// Ensure finalizer exists to prevent race conditions during
 	// deletion.
 	if !controllerutil.ContainsFinalizer(bpfmanConfig, internal.BpfmanConfigFinalizer) {
-		r.Logger.Info("Adding finalizer to Config", "name", bpfmanConfig.Name)
+		r.Logger.V(1).Info("Adding finalizer to Config", "name", bpfmanConfig.Name)
 		controllerutil.AddFinalizer(bpfmanConfig, internal.BpfmanConfigFinalizer)
 		if err := r.Update(ctx, bpfmanConfig); err != nil {
 			r.Logger.Error(err, "Failed to add finalizer to Config")
@@ -174,7 +174,7 @@ func (r *BpfmanConfigReconciler) reconcileCSIDriver(ctx context.Context, bpfmanC
 		},
 		ObjectMeta: metav1.ObjectMeta{Name: internal.BpfmanCsiDriverName},
 	}
-	r.Logger.Info("Loading object", "object", csiDriver.Name, "path", r.CsiDriverDS)
+	r.Logger.V(1).Info("Loading object", "object", csiDriver.Name, "path", r.CsiDriverDS)
 	csiDriver, err := load(csiDriver, r.CsiDriverDS, csiDriver.Name)
 	if err != nil {
 		return err
@@ -193,7 +193,7 @@ func (r *BpfmanConfigReconciler) reconcileSCC(ctx context.Context, bpfmanConfig
 				Name: internal.BpfmanRestrictedSccName,
 			},
 		}
-		r.Logger.Info("Loading object", "object", bpfmanRestrictedSCC.Name, "path", r.RestrictedSCC)
+		r.Logger.V(1).Info("Loading object", "object", bpfmanRestrictedSCC.Name, "path", r.RestrictedSCC)
 		bpfmanRestrictedSCC, err := load(bpfmanRestrictedSCC, r.RestrictedSCC, bpfmanRestrictedSCC.Name)
 		if err != nil {
 			return err
@@ -211,7 +211,7 @@ func (r *BpfmanConfigReconciler) reconcileStandardDS(ctx context.Context, bpfman
 		},
 		ObjectMeta: metav1.ObjectMeta{Name: internal.BpfmanDsName},
 	}
-	r.Logger.Info("Loading object", "object", bpfmanDS.Name, "path", r.BpfmanStandardDS)
+	r.Logger.V(1).Info("Loading object", "object", bpfmanDS.Name, "path", r.BpfmanStandardDS)
 	bpfmanDS, err := load(bpfmanDS, r.BpfmanStandardDS, bpfmanDS.Name)
 	if err != nil {
 		return err
@@ -229,7 +229,7 @@ func (r *BpfmanConfigReconciler) reconcileMetricsProxyDS(ctx context.Context, bp
 		},
 		ObjectMeta: metav1.ObjectMeta{Name: internal.BpfmanMetricsProxyDsName},
 	}
-	r.Logger.Info("Loading object", "object", metricsProxyDS.Name, "path", r.BpfmanMetricsProxyDS)
+	r.Logger.V(1).Info("Loading object", "object", metricsProxyDS.Name, "path", r.BpfmanMetricsProxyDS)
 	metricsProxyDS, err := load(metricsProxyDS, r.BpfmanMetricsProxyDS, metricsProxyDS.Name)
 	if err != nil {
 		return err
@@ -390,7 +390,7 @@ func assureResource[T client.Object](ctx context.Context, r *BpfmanConfigReconci
 		return err
 	}
 
-	r.Logger.Info("Patching object",
+	r.Logger.V(1).Info("Patching object",
 		"type", resource.GetObjectKind(), "namespace", resource.GetNamespace(), "name", resource.GetName())
 	if err := r.Patch(ctx, resource, client.Apply, client.ForceOwnership, client.FieldOwner(bpfmanConfig.Name)); err != nil {
 		r.Logger.Error(err, "Failed patching object",
@@ -405,7 +405,7 @@ func assureResource[T client.Object](ctx context.Context, r *BpfmanConfigReconci
 // removing the finalizer to allow Kubernetes garbage collection to
 // proceed via owner references.
 func (r *BpfmanConfigReconciler) handleDeletion(ctx context.Context, config *v1alpha1.Config) (ctrl.Result, error) {
-	r.Logger.Info("Config deletion requested, allowing Kubernetes garbage collection to proceed",
+	r.Logger.V(1).Info("Config deletion requested, allowing Kubernetes garbage collection to proceed",
 		"name", config.Name)
 
 	// No custom cleanup needed - all resources are managed via
@@ -417,7 +417,7 @@ func (r *BpfmanConfigReconciler) handleDeletion(ctx context.Context, config *v1a
 		return ctrl.Result{}, err
 	}
 
-	r.Logger.Info("Finalizer removed from Config, deletion will proceed", "name", config.Name)
+	r.Logger.V(1).Info("Finalizer removed from Config, deletion will proceed", "name", config.Name)
 	return ctrl.Result{}, nil
 }