From d5b8e3d61cd42d23857d7066660e03aecc89d9ce Mon Sep 17 00:00:00 2001 From: Sebastian Wagner Date: Fri, 17 Jul 2020 12:50:07 +0200 Subject: [PATCH] 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 --- src/ceph.in | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/ceph.in b/src/ceph.in index e3dc82708685..9a711d1b4c05 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) -- 2.47.3