From cb748ccb2264aea93dd4717a46d7f833c5b1eccd Mon Sep 17 00:00:00 2001 From: Andrew Schoen Date: Thu, 18 Jan 2018 11:10:48 -0600 Subject: [PATCH] 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 --- src/ceph-volume/ceph_volume/devices/lvm/zap.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/ceph-volume/ceph_volume/devices/lvm/zap.py b/src/ceph-volume/ceph_volume/devices/lvm/zap.py index 537c0cddda270..f7ec1116934d1 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) -- 2.39.5