fix: fix ensureDeleted of openshiftSccs via returning an error

The func does not return an error if there is an error and always return
nil. Fix the same via returning and error if there non-expected failure.

Signed-off-by: Nitin Goyal <nigoyal@redhat.com>

controllers/scc.go

@@ -73,12 +73,11 @@ func (c openshiftSccs) ensureDeleted(r *LVMClusterReconciler, ctx context.Contex
 		for _, scc := range sccs {
 			err = r.SecurityClient.SecurityContextConstraints().Delete(ctx, scc.Name, metav1.DeleteOptions{})
 			if err != nil {
-				if errors.IsNotFound(err) {
-					r.Log.Info("SecurityContextConstraint is already deleted", "SecurityContextConstraint", scc.Name)
-					return nil
-				} else {
+				if !errors.IsNotFound(err) {
 					r.Log.Error(err, "failed to delete SecurityContextConstraint", "SecurityContextConstraint", scc.Name)
+					return err
 				}
+				r.Log.Info("SecurityContextConstraint is already deleted", "SecurityContextConstraint", scc.Name)
 			}
 		}
 	}