chore: added a few debug log (cloudnative-pg#1068)

Add some debug logging when finding conflicts during update,
patch or creation of an object.

Close cloudnative-pg#1065

Signed-off-by: Jonathan Gonzalez V <jonathan.gonzalez@enterprisedb.com>
Signed-off-by: Gabriele Bartolini <gabriele.bartolini@enterprisedb.com>
Co-authored-by: Gabriele Bartolini <gabriele.bartolini@enterprisedb.com>

Author: sxd

controllers/cluster_controller.go

@@ -217,6 +217,7 @@ func (r *ClusterReconciler) reconcile(ctx context.Context, cluster *apiv1.Cluste
 		if apierrs.IsConflict(err) {
 			// Requeue a new reconciliation cycle, as in this point we need
 			// to quickly react the changes
+			contextLogger.Debug("Conflict error while reconciling resource status", "error", err)
 			return ctrl.Result{Requeue: true}, nil
 		}
 
@@ -239,6 +240,8 @@ func (r *ClusterReconciler) reconcile(ctx context.Context, cluster *apiv1.Cluste
 	// we update all the cluster status fields that require the instances status
 	if err := r.updateClusterStatusThatRequiresInstancesState(ctx, cluster, instancesStatus); err != nil {
 		if apierrs.IsConflict(err) {
+			contextLogger.Debug("Conflict error while reconciling cluster status nad instance state",
+				"error", err)
 			return ctrl.Result{Requeue: true}, nil
 		}
 		return ctrl.Result{}, fmt.Errorf("cannot update the instances status on the cluster: %w", err)
@@ -297,6 +300,7 @@ func (r *ClusterReconciler) reconcile(ctx context.Context, cluster *apiv1.Cluste
 		if apierrs.IsConflict(err) {
 			// Requeue a new reconciliation cycle, as in this point we need
 			// to quickly react the changes
+			contextLogger.Debug("Conflict error while reconciling online update", "error", err)
 			return ctrl.Result{Requeue: true}, nil
 		}
 
@@ -481,6 +485,7 @@ func (r *ClusterReconciler) reconcileResources(
 	// Reconcile PVC resource requirements
 	if err := r.ReconcilePVCs(ctx, cluster, resources); err != nil {
 		if apierrs.IsConflict(err) {
+			contextLogger.Debug("Conflict error while reconciling PVCs", "error", err)
 			return ctrl.Result{Requeue: true}, nil
 		}
 		return ctrl.Result{}, err
@@ -525,6 +530,7 @@ func (r *ClusterReconciler) deleteEvictedPods(ctx context.Context, cluster *apiv
 				"podStatus", resources.instances.Items[idx].Status)
 			if err := r.Delete(ctx, &resources.instances.Items[idx]); err != nil {
 				if apierrs.IsConflict(err) {
+					contextLogger.Debug("Conflict error while deleting instances item", "error", err)
 					return &ctrl.Result{Requeue: true}, nil
 				}
 				return nil, err

controllers/pooler_controller.go

@@ -110,6 +110,7 @@ func (r *PoolerReconciler) Reconcile(ctx context.Context, req ctrl.Request) (ctr
 		if apierrs.IsConflict(err) {
 			// Requeue a reconciliation loop since the resource
 			// changed while we were synchronizing it
+			contextLogger.Debug("Conflict while reconciling pooler status", "error", err)
 			return ctrl.Result{Requeue: true}, nil
 		}
 	}

controllers/scheduledbackup_controller.go

@@ -142,6 +142,7 @@ func ReconcileScheduledBackup(
 		if err != nil {
 			if apierrs.IsConflict(err) {
 				// Retry later, the cache is stale
+				contextLogger.Debug("Conflict while reconciling the scheduled backup", "error", err)
 				return ctrl.Result{}, nil
 			}
 			return ctrl.Result{}, err
@@ -218,6 +219,7 @@ func createBackup(
 	if err := client.Create(ctx, backup); err != nil {
 		if apierrs.IsConflict(err) {
 			// Retry later, the cache is stale
+			contextLogger.Debug("Conflict while creating backup", "error", err)
 			return ctrl.Result{}, nil
 		}
 
@@ -243,6 +245,7 @@ func createBackup(
 	if err := client.Status().Update(ctx, scheduledBackup); err != nil {
 		if apierrs.IsConflict(err) {
 			// Retry later, the cache is stale
+			contextLogger.Debug("Conflict while updating scheduled backup", "error", err)
 			return ctrl.Result{}, nil
 		}
 		return ctrl.Result{}, err