Re: [PATCH] Don't let lvremove failures from incomplete vgs crash the install. (#497401)

[Chris Lumens <clumens redhat com>]

>  storage/devicetree.py |    8 +++++++-
>  1 files changed, 7 insertions(+), 1 deletions(-)
> 
> diff --git a/storage/devicetree.py b/storage/devicetree.py
> index 996e702..2c259ec 100644
> --- a/storage/devicetree.py
> +++ b/storage/devicetree.py
> @@ -1558,7 +1558,13 @@ class DeviceTree(object):
>  
>                      # destroy all lvs.
>                      for lv in device.vg.lvs:
> -                        lv.destroy()
> +                        try:
> +                            # reinitializeVG should clean up if necessary
> +                            lv.destroy()
> +                        except StorageError as e:
> +                            log.info("error removing lv %s from "
> +                                     "inconsistent/incomplete vg %s"
> +                                     % (lv.lvname, device.vg.name))
>                          device.vg._removeLogVol(lv)
>                          self._removeDevice(lv)

Looks good, given that we're just going to remove everything after this
anyway.

- Chris