Skip to content

Commit

Permalink
fix: can not delete lvmcluster with wrong disks path in CR
Browse files Browse the repository at this point in the history
in this commit we are updating the node status when lvmd config file
does not exists.

Signed-off-by: riya-singhal31 <rsinghal@redhat.com>
  • Loading branch information
riya-singhal31 committed Sep 29, 2022
1 parent 282bcbc commit 9e0b635
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions pkg/vgmanager/vgmanager_controller.go
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit 9e0b635

Please sign in to comment.