fix: deletion of lvmcluster which includes wrong paths

Deleting the lvmvolumegroup CR does not clean the VG entry in the lvmvolumegroupnodestatus if lvmdconfig file does not exists and returns immediately. Fix the same via updating the lvmvolumegroupnodestatus. Signed-off-by: riya-singhal31 <[email protected]>
openshift · Sep 30, 2022 · d04209b · d04209b
1 parent 282bcbc
commit d04209b
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