From: David Zafman <dzafman@redhat.com>
Date: Wed, 6 Dec 2017 01:53:01 +0000 (-0800)
Subject: ceph-objectstore-tool: Improve ceph-objectstore-tool usage output
X-Git-Tag: v13.0.1~12^2~4
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=c6896cccc35bc9a159bbe8c3bbb6a0c9ab08fcac;p=ceph.git

ceph-objectstore-tool: Improve ceph-objectstore-tool usage output

Remove unused argument test-align

Caused by: 1c8731c31eb53e17d8f363906438963ab5688fe0

Positional arguments are described by usage() because
the parsing code creates descriptions like "--object" for the
object positional argument.

Signed-off-by: David Zafman <dzafman@redhat.com>
---

diff --git a/src/tools/ceph_objectstore_tool.cc b/src/tools/ceph_objectstore_tool.cc
index 45346b49a2194..a7ec4053910c1 100644
--- a/src/tools/ceph_objectstore_tool.cc
+++ b/src/tools/ceph_objectstore_tool.cc
@@ -292,7 +292,6 @@ int file_fd = fd_none;
 bool debug;
 bool force = false;
 super_header sh;
-uint64_t testalign;
 
 static int get_fd_data(int fd, bufferlist &bl)
 {
@@ -2640,10 +2639,9 @@ int main(int argc, char **argv)
     ("arg1", po::value<string>(&arg1), "arg1 based on cmd, "
      "for apply-layout-settings: target hash level split to")
     ("arg2", po::value<string>(&arg2), "arg2 based on cmd")
-    ("test-align", po::value<uint64_t>(&testalign)->default_value(0), "hidden align option for testing")
     ;
 
-  po::options_description all("All options");
+  po::options_description all;
   all.add(desc).add(positional);
 
   po::positional_options_description pd;
@@ -2664,7 +2662,7 @@ int main(int argc, char **argv)
   }
 
   if (vm.count("help")) {
-    usage(all);
+    usage(desc);
     return 1;
   }