From: Sebastian Wagner <sebastian.wagner@suse.com>
Date: Fri, 17 Jul 2020 10:50:07 +0000 (+0200)
Subject: ceph: Fix `tell` for cephadm
X-Git-Tag: v16.1.0~1684^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=d5b8e3d61cd42d23857d7066660e03aecc89d9ce;p=ceph.git

ceph: Fix `tell` for cephadm

The tell target might contain dots now. Like
`mds.myfs.myhost.rnusmq` is now a valid
target.

Fixes: https://tracker.ceph.com/issues/46560

Signed-off-by: Sebastian Wagner <sebastian.wagner@suse.com>
---

diff --git a/src/ceph.in b/src/ceph.in
index e3dc827086854..9a711d1b4c059 100755
--- a/src/ceph.in
+++ b/src/ceph.in
@@ -1038,7 +1038,7 @@ def main():
     if parsed_args.help:
         target = None
         if len(childargs) >= 2 and childargs[0] == 'tell':
-            target = childargs[1].split('.')
+            target = childargs[1].split('.', 1)
             if not validate_target(target):
                 print('target {0} doesn\'t exist; please pass correct target to tell command (e.g., mon.a, osd.1, mds.a, mgr)'.format(childargs[1]), file=sys.stderr)
                 return 1
@@ -1054,7 +1054,7 @@ def main():
 
     # implement "tell service.id help"
     if len(childargs) >= 3 and childargs[0] == 'tell' and childargs[2] == 'help':
-        target = childargs[1].split('.')
+        target = childargs[1].split('.', 1)
         if validate_target(target):
             hdr('Tell %s commands' % target[0])
             return do_extended_help(parser, childargs, target, None)