update

controller/linodemachine_controller.go

@@ -173,12 +173,15 @@ func (r *LinodeMachineReconciler) reconcile(ctx context.Context, logger logr.Log
 	failureReason := cerrs.MachineStatusError("UnknownError")
 	//nolint:dupl // Code duplication is simplicity in this case.
 	defer func() {
-		if err != nil && !util.IsRetryableError(err) {
-			machineScope.LinodeMachine.Status.FailureReason = util.Pointer(failureReason)
-			machineScope.LinodeMachine.Status.FailureMessage = util.Pointer(err.Error())
-
-			conditions.MarkFalse(machineScope.LinodeMachine, clusterv1.ReadyCondition, string(failureReason), clusterv1.ConditionSeverityError, "%s", err.Error())
+		if err != nil {
+			// Only set failure reason if the error is not retryable.
+			if !util.IsRetryableError(err) {
+				machineScope.LinodeMachine.Status.FailureReason = util.Pointer(failureReason)
+				machineScope.LinodeMachine.Status.FailureMessage = util.Pointer(err.Error())
+				conditions.MarkFalse(machineScope.LinodeMachine, clusterv1.ReadyCondition, string(failureReason), clusterv1.ConditionSeverityError, "%s", err.Error())
+			}
 
+			// Record the event regardless of whether the error is retryable or not for visibility.
 			r.Recorder.Event(machineScope.LinodeMachine, corev1.EventTypeWarning, string(failureReason), err.Error())
 		}