From b8ee4efec3f475268c4b5ea8c397c7be05370497 Mon Sep 17 00:00:00 2001
From: Alfredo Deza <adeza@redhat.com>
Date: Wed, 12 Jul 2017 12:15:42 -0400
Subject: [PATCH] ceph-volume: lvm: use argparse, report help as early as
 possible

Signed-off-by: Alfredo Deza <adeza@redhat.com>
---
 .../ceph_volume/devices/lvm/activate.py       |  2 ++
 .../ceph_volume/devices/lvm/main.py           | 19 +++++++++----------
 .../ceph_volume/devices/lvm/prepare.py        |  4 +++-
 3 files changed, 14 insertions(+), 11 deletions(-)

diff --git a/src/ceph-volume/ceph_volume/devices/lvm/activate.py b/src/ceph-volume/ceph_volume/devices/lvm/activate.py
index 9ed80dbda0b1..b6d04f52d038 100644
--- a/src/ceph-volume/ceph_volume/devices/lvm/activate.py
+++ b/src/ceph-volume/ceph_volume/devices/lvm/activate.py
@@ -99,4 +99,6 @@ class Activate(object):
             help='filestore objectstore (current default)',
         )
         args = parser.parse_args(self.argv[1:])
+        if len(self.argv) <= 1:
+            return parser.print_help()
         self.activate(args)
diff --git a/src/ceph-volume/ceph_volume/devices/lvm/main.py b/src/ceph-volume/ceph_volume/devices/lvm/main.py
index 98984d3c4f76..e9cf9cff4294 100644
--- a/src/ceph-volume/ceph_volume/devices/lvm/main.py
+++ b/src/ceph-volume/ceph_volume/devices/lvm/main.py
@@ -1,5 +1,6 @@
+import argparse
 from textwrap import dedent
-from tambo import Transport
+from ceph_volume import terminal
 import activate
 import prepare
 
@@ -26,14 +27,12 @@ class LVM(object):
         return self._help.format(sub_help=sub_help)
 
     def main(self):
-        options = [['--log', '--logging']]
-        parser = Transport(self.argv, mapper=self.mapper,
-                           options=options, check_help=False,
-                           check_version=False)
-        parser.parse_args()
-        parser.catch_help = self.print_help(parser.subhelp())
-        parser.mapper = self.mapper
+        terminal.dispatch(self.mapper, self.argv)
+        parser = argparse.ArgumentParser(
+            prog='ceph-volume lvm',
+            formatter_class=argparse.RawDescriptionHelpFormatter,
+            description=self.print_help(terminal.subhelp(self.mapper)),
+        )
+        parser.parse_args(self.argv)
         if len(self.argv) <= 1:
             return parser.print_help()
-        parser.dispatch()
-        parser.catches_help()
diff --git a/src/ceph-volume/ceph_volume/devices/lvm/prepare.py b/src/ceph-volume/ceph_volume/devices/lvm/prepare.py
index ef63e5ea1900..46b10b52306d 100644
--- a/src/ceph-volume/ceph_volume/devices/lvm/prepare.py
+++ b/src/ceph-volume/ceph_volume/devices/lvm/prepare.py
@@ -206,5 +206,7 @@ class Prepare(object):
             '--osd-fsid',
             help='Reuse an existing OSD fsid',
         )
-        args = parser.parse_args(self.argv[1:])
+        args = parser.parse_args(self.argv)
+        if len(self.argv) <= 1:
+            return parser.print_help()
         self.prepare(args)
-- 
2.47.3