Some minor changes (comments, etc) to improve readability

Signed-off-by: Michael Shitrit <mshitrit@redhat.com>

Author: mshitrit

controllers/machinehealthcheck_controller.go

@@ -404,17 +404,15 @@ func (r *MachineHealthCheckReconciler) remediate(target resources.Target, rm res
 		return errors.Wrapf(err, "failed to get remediation template")
 	}
 
-	// TODO add control plane label
-
-	// create remediation CR
 	var nodeNamePtr *string
-	nodeName := ""
 	if target.Node != nil && target.Node.ResourceVersion != "" {
 		nodeNamePtr = &target.Node.Name
-		nodeName = target.Node.Name
 	}
 
-	remediationCR, err := rm.GenerateRemediationCRForMachine(target.Machine, target.MHC, template, nodeName)
+	// TODO add control plane label
+
+	// create remediation CR
+	remediationCR, err := rm.GenerateRemediationCRForMachine(target.Machine, target.MHC, template, pointer.StringDeref(nodeNamePtr, ""))
 	if err != nil {
 		return errors.Wrapf(err, "failed to generate remediation CR")
 	}

controllers/resources/manager.go

@@ -127,6 +127,7 @@ func (m *manager) generateRemediationCR(name string, nodeName string, healthChec
 	templateSpec, _, _ := unstructured.NestedMap(template.Object, "spec", "template", "spec")
 	unstructured.SetNestedField(remediationCR.Object, templateSpec, "spec")
 
+	// Multiple same kind templates are never supported for MHC, and remediators are not expected to handle generated names in this case, even if they do for NHC.
 	isMHCRemediation := name != nodeName
 	if annotations.HasMultipleTemplatesAnnotation(template) && !isMHCRemediation {
 		remediationCR.SetGenerateName(fmt.Sprintf("%s-", name))