improve logging during Machine deletion

Author: schrej

internal/controllers/machine/machine_controller.go

@@ -368,8 +368,6 @@ func (r *Reconciler) reconcileDelete(ctx context.Context, cluster *clusterv1.Clu
 		// After node draining is completed, and if isNodeVolumeDetachingAllowed returns True, make sure all
 		// volumes are detached before proceeding to delete the Node.
 		if r.isNodeVolumeDetachingAllowed(m) {
-			log.Info("Waiting for node volumes to be detached", "Node", klog.KRef("", m.Status.NodeRef.Name))
-
 			// The VolumeDetachSucceededCondition never exists before we wait for volume detachment for the first time,
 			// so its transition time can be used to record the first time we wait for volume detachment.
 			// This `if` condition prevents the transition time to be changed more than once.
@@ -412,10 +410,16 @@ func (r *Reconciler) reconcileDelete(ctx context.Context, cluster *clusterv1.Clu
 	}
 
 	if ok, err := r.reconcileDeleteInfrastructure(ctx, m); !ok || err != nil {
+		if err == nil {
+			log.Info("Waiting for infrastructure to be deleted", "Infrastructure", klog.KRef("", m.Spec.InfrastructureRef.Name))
+		}
 		return ctrl.Result{}, err
 	}
 
 	if ok, err := r.reconcileDeleteBootstrap(ctx, m); !ok || err != nil {
+		if err == nil {
+			log.Info("Waiting for bootstrap to be deleted", "Bootstrap", klog.KRef("", m.Spec.Bootstrap.ConfigRef.Name))
+		}
 		return ctrl.Result{}, err
 	}