mon: add missing format argument to crush dump

Signed-off-by: Loic Dachary <loic@dachary.org>

diff --git a/src/mon/MonCommands.h b/src/mon/MonCommands.h
index 176b17772601f5a84ad9ad1e1618e7f498ed06ca..60fa9815524e83082123baa32f4060ce9cab4b06 100644 (file)
--- a/src/mon/MonCommands.h
+++ b/src/mon/MonCommands.h
@@ -363,11 +363,14 @@ COMMAND("osd lspools " \
 COMMAND("osd blacklist ls", "show blacklisted clients", "osd", "r", "cli,rest")
 COMMAND("osd crush rule list", "list crush rules", "osd", "r", "cli,rest")
 COMMAND("osd crush rule ls", "list crush rules", "osd", "r", "cli,rest")
-COMMAND("osd crush dump", "dump crush map", "osd", "r", "cli,rest")
 COMMAND("osd crush rule dump " \
        "name=format,type=CephChoices,strings=json|json-pretty|xml|xml-pretty,req=false", \
        "dump crush rules formatted as <format> (default json-pretty)", \
        "osd", "r", "cli,rest")
+COMMAND("osd crush dump " \
+       "name=format,type=CephChoices,strings=json|json-pretty|xml|xml-pretty,req=false", \
+       "dump crush map formatted as <format> (default json-pretty)", \
+       "osd", "r", "cli,rest")
 COMMAND("osd setcrushmap", "set crush map from input file", \
        "osd", "rw", "cli,rest")
 COMMAND("osd crush set", "set crush map from input file", \

diff --git a/src/test/pybind/test_ceph_argparse.py b/src/test/pybind/test_ceph_argparse.py
index e10d1aa2baefe39fdac80a3175f0c4f3ee98f3f7..822a430ee389550d5623c59cddc07369dd800dc4 100755 (executable)
--- a/src/test/pybind/test_ceph_argparse.py
+++ b/src/test/pybind/test_ceph_argparse.py
@@ -585,9 +585,12 @@ class TestOSD(TestArgparse):
 
     def test_crush_dump(self):
         self.assert_valid_command(['osd', 'crush', 'dump'])
+        for format in ('json', 'json-pretty', 'xml', 'xml-pretty'):
+                       self.assert_valid_command(['osd', 'crush', 'dump', format])
         assert_equal({}, validate_command(sigdict, ['osd', 'crush']))
         assert_equal({}, validate_command(sigdict, ['osd', 'crush',
-                                                    'dump', 'toomany']))
+                                                    'dump', 'json',
+                                                    'toomany']))
 
     def test_setcrushmap(self):
         self.check_no_arg('osd', 'setcrushmap')