From: Alfredo Deza <adeza@redhat.com>
Date: Fri, 7 Dec 2018 12:53:43 +0000 (-0500)
Subject: ceph-volume lvm.zap update success message for OSD IDs
X-Git-Tag: v12.2.11~10^2~3
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=bdc9783f19786e9990273108a2edf7cf41e54686;p=ceph.git

ceph-volume lvm.zap update success message for OSD IDs

Signed-off-by: Alfredo Deza <adeza@redhat.com>
(cherry picked from commit 73bb17f79a5b67b28ceace9dc7dcc2dde38dad5e)
---

diff --git a/src/ceph-volume/ceph_volume/devices/lvm/zap.py b/src/ceph-volume/ceph_volume/devices/lvm/zap.py
index fbd784b63faa..328a036152e2 100644
--- a/src/ceph-volume/ceph_volume/devices/lvm/zap.py
+++ b/src/ceph-volume/ceph_volume/devices/lvm/zap.py
@@ -69,7 +69,7 @@ def ensure_associated_lvs(lvs):
     are LVs or partitions, so that they can be accurately reported.
     """
     # look for many LVs for each backing type, because it is possible to
-    # recieve a filtering for osd.1, and have multiple failed deployments
+    # receive a filtering for osd.1, and have multiple failed deployments
     # leaving many journals with osd.1 - usually, only a single LV will be
     # returned
     journal_lvs = lvs._filter(lv_tags={'ceph.type': 'journal'})
@@ -238,7 +238,14 @@ class Zap(object):
             if device.is_device:
                 self.zap_raw_device(device)
 
-        terminal.success("Zapping successful for: %s" % ", ".join([str(d) for d in self.args.devices]))
+        if self.args.devices:
+            terminal.success(
+                "Zapping successful for: %s" % ", ".join([str(d) for d in self.args.devices])
+            )
+        else:
+            terminal.success(
+                "Zapping successful for OSD: %s" % self.args.osd_id or self.args.osd_fsid
+            )
 
     @decorators.needs_root
     def zap_osd(self):