fix: deleting lvmcluster successfully in case of wrong disk path in CR

lvmcluster was failing to delete in case of wrong disk path provided in spec. Updated the code to update the status of lvmvolumegroupnodestatus in case when there is no lvmd file exists like in case when the user gives wrong disk path in lvmclusterCR, because lvmclusterCR reads the information from lvmvolumegroupnodestatus. Signed-off-by: riya-singhal31 <[email protected]>
openshift · Sep 29, 2022 · ac5a1ec · ac5a1ec
1 parent 282bcbc
commit ac5a1ec
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/pkg/vgmanager/vgmanager_controller.go b/pkg/vgmanager/vgmanager_controller.go
@@ -324,6 +324,10 @@ func (r *VGReconciler) processDelete(ctx context.Context, volumeGroup *lvmv1alph
 	}
 	if lvmdConfig == nil {
 		r.Log.Info("lvmd config file does not exist")
+		if statuserr := r.updateStatus(ctx, nil); statuserr != nil {
+			r.Log.Error(statuserr, "failed to update status", "VGName", volumeGroup.Name)
+			return statuserr
+		}
 		return nil
 	}
 	// To avoid having to iterate through device classes multiple times, map from name to config index