Provide a little more clarity on status error messages

CrowdStrike · Jun 13, 2022 · 5280cc7 · 5280cc7
1 parent 51e4dad
commit 5280cc7
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/controllers/falcon/falconnodesensor_controller.go b/controllers/falcon/falconnodesensor_controller.go
@@ -261,7 +261,7 @@ func (r *FalconNodeSensorReconciler) Reconcile(ctx context.Context, req ctrl.Req
 		nodesensor.Status.Sensor = &strings.Split(image, ":")[1]
 		err = r.Status().Update(ctx, nodesensor)
 		if err != nil {
-			log.Error(err, "Failed to update FalconNodeSensor status")
+			log.Error(err, "Failed to update FalconNodeSensor status for nodesensor.Status.Sensor")
 			return ctrl.Result{}, err
 		}
 	}
@@ -270,7 +270,7 @@ func (r *FalconNodeSensorReconciler) Reconcile(ctx context.Context, req ctrl.Req
 		nodesensor.Status.Version = version.Get()
 		err = r.Status().Update(ctx, nodesensor)
 		if err != nil {
-			log.Error(err, "Failed to update FalconNodeSensor status")
+			log.Error(err, "Failed to update FalconNodeSensor status for nodesensor.Status.Version")
 			return ctrl.Result{}, err
 		}
 	}
@@ -683,7 +683,7 @@ func (r *FalconNodeSensorReconciler) conditionsUpdate(condType string, status me
 	})
 
 	if err := r.Status().Update(ctx, nodesensor); err != nil {
-		logger.Error(err, "Failed to update FalconNodeSensor status")
+		logger.Error(err, "Failed to update FalconNodeSensor status", "Failed to update the Condition at Reasoning", reason)
 		return err
 	}