Allow external experimental remediation even if there's no controller…

… owner.

Baremetal external experimental remediation plugs into MHC with an annotation as per https://github.com/openshift/enhancements/blob/master/enhancements/machine-api/machine-health-checking.md#out-of-tree-experimental-remediation-controller-eg-baremetal-reboot

We want it to apply even before checking the machines have an owner controller so it covers more scenarios and delegate all the responsibility on the remediaton system per https://github.com/openshift/cluster-api-provider-baremetal/blob/master/docs/remediation.md#assumptions

This will allow external remediation controller to remediate baremetal Masters which currently
don't have any controller owner.

Signed-off-by: Nir <niry@redhat.com>

pkg/controller/machinehealthcheck/machinehealthcheck_controller.go

@@ -435,10 +435,6 @@ func (r *ReconcileMachineHealthCheck) mhcRequestsFromMachine(o handler.MapObject
 
 func (t *target) remediate(r *ReconcileMachineHealthCheck) error {
 	glog.Infof(" %s: start remediation logic", t.string())
-	if !t.hasControllerOwner() {
-		glog.Infof("%s: no controller owner, skipping remediation", t.string())
-		return nil
-	}
 
 	remediationStrategy, ok := t.MHC.Annotations[remediationStrategyAnnotation]
 	if ok {
@@ -447,6 +443,11 @@ func (t *target) remediate(r *ReconcileMachineHealthCheck) error {
 		}
 	}
 
+	if !t.hasControllerOwner() {
+		glog.Infof("%s: no controller owner, skipping remediation", t.string())
+		return nil
+	}
+
 	glog.Infof("%s: deleting", t.string())
 	if err := r.client.Delete(context.TODO(), &t.Machine); err != nil {
 		r.recorder.Eventf(