Remove deletion timestamp check from NHC timeout logic, as it would cause a bug which will prevent SNR from removing the finilizer on an orphaned CR

Signed-off-by: Michael Shitrit <[email protected]>

Author: mshitrit

controllers/selfnoderemediation_controller.go

@@ -807,7 +807,7 @@ func (r *SelfNodeRemediationReconciler) removeNoExecuteTaint(node *v1.Node) erro
 }
 
 func (r *SelfNodeRemediationReconciler) isStoppedByNHC(snr *v1alpha1.SelfNodeRemediation) bool {
-	if snr != nil && snr.Annotations != nil && snr.DeletionTimestamp == nil {
+	if snr != nil && snr.Annotations != nil {
 		_, isTimeoutIssued := snr.Annotations[nhcTimeOutAnnotation]
 		return isTimeoutIssued
 	}