From: Andrew Schoen Date: Thu, 18 Jan 2018 17:10:48 +0000 (-0600) Subject: ceph-volume: also remove pvs with lvm zap --destroy X-Git-Tag: v13.0.2~453^2~6 X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=cb748ccb2264aea93dd4717a46d7f833c5b1eccd;p=ceph-ci.git 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 --- diff --git a/src/ceph-volume/ceph_volume/devices/lvm/zap.py b/src/ceph-volume/ceph_volume/devices/lvm/zap.py index 537c0cddda2..f7ec1116934 100644 --- 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)