ceph-volume: also remove pvs with lvm zap --destroy

Leaving the pv around doesn't keep us from redeploying on
that device because we also wipefs, but explicitly destroying
it makes the output read better.

Signed-off-by: Andrew Schoen <aschoen@redhat.com>

diff --git a/src/ceph-volume/ceph_volume/devices/lvm/zap.py b/src/ceph-volume/ceph_volume/devices/lvm/zap.py
index 537c0cddda27083aeaa2ba847a0435935d916b1b..f7ec1116934d1170ea4e9f0a192a3fd4256ff4d6 100644 (file)
--- a/src/ceph-volume/ceph_volume/devices/lvm/zap.py
+++ b/src/ceph-volume/ceph_volume/devices/lvm/zap.py
@@ -69,6 +69,9 @@ class Zap(object):
                 logger.info("Destroying volume group %s because --destroy was given", vg_name)
                 terminal.write("Destroying volume group %s because --destroy was given" % vg_name)
                 api.remove_vg(vg_name)
+                logger.info("Destroying physical volume %s because --destroy was given", device)
+                terminal.write("Destroying physical volume %s because --destroy was given" % device)
+                api.remove_pv(device)
 
         wipefs(path)
         zap_data(path)