From: Jesse Williamson Date: Tue, 23 Jan 2018 13:13:47 +0000 (-0800) Subject: mon: update MonCommands X-Git-Tag: v13.1.0~499^2~1 X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=a24a6ccde2129af898663ee479c52bbcc87ac2c0;p=ceph.git mon: update MonCommands Regardless of the "inner" implementation, at this level we officially make all remove commands use "rm" and not "remove", "del", or "delete". Deprecates "remove", "del", and "delete". Adjusts code so that "rm" preceeds "remove". Signed-off-by: Jesse Williamson --- diff --git a/src/mon/MonCommands.h b/src/mon/MonCommands.h index 0f1729d4f9a2..7da9fb7973a5 100644 --- a/src/mon/MonCommands.h +++ b/src/mon/MonCommands.h @@ -171,10 +171,11 @@ COMMAND("auth caps " \ "name=caps,type=CephString,n=N", \ "update caps for from caps specified in the command", \ "auth", "rwx", "cli,rest") -COMMAND("auth del " \ +COMMAND_WITH_FLAG("auth del " \ "name=entity,type=CephString", \ "delete all caps for ", \ - "auth", "rwx", "cli,rest") + "auth", "rwx", "cli,rest", \ + FLAG(DEPRECATED)) COMMAND("auth rm " \ "name=entity,type=CephString", \ "remove all caps for ", \ @@ -348,10 +349,10 @@ COMMAND("mds compat rm_incompat " \ COMMAND_WITH_FLAG("mds add_data_pool " \ "name=pool,type=CephString", \ "add data pool ", "mds", "rw", "cli,rest", FLAG(OBSOLETE)) -COMMAND_WITH_FLAG("mds remove_data_pool " \ +COMMAND_WITH_FLAG("mds rm_data_pool " \ "name=pool,type=CephString", \ "remove data pool ", "mds", "rw", "cli,rest", FLAG(OBSOLETE)) -COMMAND_WITH_FLAG("mds rm_data_pool " \ +COMMAND_WITH_FLAG("mds remove_data_pool " \ "name=pool,type=CephString", \ "remove data pool ", "mds", "rw", "cli,rest", FLAG(OBSOLETE)) COMMAND_WITH_FLAG("mds newfs " \ @@ -426,12 +427,13 @@ COMMAND("mon add " \ "name=name,type=CephString " \ "name=addr,type=CephIPAddr", \ "add new monitor named at ", "mon", "rw", "cli,rest") -COMMAND("mon remove " \ - "name=name,type=CephString", \ - "remove monitor named ", "mon", "rw", "cli,rest") COMMAND("mon rm " \ "name=name,type=CephString", \ "remove monitor named ", "mon", "rw", "cli,rest") +COMMAND_WITH_FLAG("mon remove " \ + "name=name,type=CephString", \ + "remove monitor named ", "mon", "rw", "cli,rest", \ + FLAG(DEPRECATED)) COMMAND("mon feature ls " \ "name=with_value,type=CephChoices,strings=--with-value,req=false", \ "list available mon map features to be set/unset", \ @@ -585,11 +587,12 @@ COMMAND("osd crush rm " \ "name=ancestor,type=CephString,req=false,goodchars=[A-Za-z0-9-_.]", \ "remove from crush map (everywhere, or just at )",\ "osd", "rw", "cli,rest") -COMMAND("osd crush remove " \ +COMMAND_WITH_FLAG("osd crush remove " \ "name=name,type=CephString,goodchars=[A-Za-z0-9-_.] " \ "name=ancestor,type=CephString,req=false,goodchars=[A-Za-z0-9-_.]", \ "remove from crush map (everywhere, or just at )", \ - "osd", "rw", "cli,rest") + "osd", "rw", "cli,rest", \ + FLAG(DEPRECATED)) COMMAND("osd crush unlink " \ "name=name,type=CephString,goodchars=[A-Za-z0-9-_.] " \ "name=ancestor,type=CephString,req=false,goodchars=[A-Za-z0-9-_.]", \ @@ -914,12 +917,13 @@ COMMAND("osd pool create " \ "name=rule,type=CephString,req=false " \ "name=expected_num_objects,type=CephInt,req=false", \ "create pool", "osd", "rw", "cli,rest") -COMMAND("osd pool delete " \ +COMMAND_WITH_FLAG("osd pool delete " \ "name=pool,type=CephPoolname " \ "name=pool2,type=CephPoolname,req=false " \ "name=sure,type=CephString,req=false", \ "delete pool", \ - "osd", "rw", "cli,rest") + "osd", "rw", "cli,rest", \ + FLAG(DEPRECATED)) COMMAND("osd pool rm " \ "name=pool,type=CephPoolname " \ "name=pool2,type=CephPoolname,req=false " \ @@ -994,16 +998,17 @@ COMMAND("osd tier add " \ "name=force_nonempty,type=CephChoices,strings=--force-nonempty,req=false", "add the tier (the second one) to base pool (the first one)", \ "osd", "rw", "cli,rest") -COMMAND("osd tier remove " \ +COMMAND("osd tier rm " \ "name=pool,type=CephPoolname " \ "name=tierpool,type=CephPoolname", "remove the tier (the second one) from base pool (the first one)", \ "osd", "rw", "cli,rest") -COMMAND("osd tier rm " \ +COMMAND_WITH_FLAG("osd tier remove " \ "name=pool,type=CephPoolname " \ "name=tierpool,type=CephPoolname", "remove the tier (the second one) from base pool (the first one)", \ - "osd", "rw", "cli,rest") + "osd", "rw", "cli,rest", \ + FLAG(DEPRECATED)) COMMAND("osd tier cache-mode " \ "name=pool,type=CephPoolname " \ "name=mode,type=CephChoices,strings=none|writeback|forward|readonly|readforward|proxy|readproxy " \ @@ -1013,12 +1018,13 @@ COMMAND("osd tier set-overlay " \ "name=pool,type=CephPoolname " \ "name=overlaypool,type=CephPoolname", \ "set the overlay pool for base pool to be ", "osd", "rw", "cli,rest") -COMMAND("osd tier remove-overlay " \ - "name=pool,type=CephPoolname ", \ - "remove the overlay pool for base pool ", "osd", "rw", "cli,rest") COMMAND("osd tier rm-overlay " \ "name=pool,type=CephPoolname ", \ "remove the overlay pool for base pool ", "osd", "rw", "cli,rest") +COMMAND_WITH_FLAG("osd tier remove-overlay " \ + "name=pool,type=CephPoolname ", \ + "remove the overlay pool for base pool ", "osd", "rw", "cli,rest", \ + FLAG(DEPRECATED)) COMMAND("osd tier add-cache " \ "name=pool,type=CephPoolname " \ @@ -1043,9 +1049,10 @@ COMMAND_WITH_FLAG("config-key put " \ "name=val,type=CephString,req=false", \ "put , value ", "config-key", "rw", "cli,rest", FLAG(DEPRECATED)) -COMMAND("config-key del " \ +COMMAND_WITH_FLAG("config-key del " \ "name=key,type=CephString", \ - "delete ", "config-key", "rw", "cli,rest") + "delete ", "config-key", "rw", "cli,rest", \ + FLAG(DEPRECATED)) COMMAND("config-key rm " \ "name=key,type=CephString", \ "rm ", "config-key", "rw", "cli,rest")