From: Nathan Cutler <ncutler@suse.com>
Date: Wed, 6 Nov 2019 07:00:45 +0000 (+0100)
Subject: qa/tasks/ceph.py/create_simple_monmap: use split_role
X-Git-Tag: v13.2.7~8^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F31424%2Fhead;p=ceph.git

qa/tasks/ceph.py/create_simple_monmap: use split_role

Signed-off-by: Nathan Cutler <ncutler@suse.com>
---

diff --git a/qa/tasks/ceph.py b/qa/tasks/ceph.py
index 0fcc2cc90104..49cf6473c89d 100644
--- a/qa/tasks/ceph.py
+++ b/qa/tasks/ceph.py
@@ -443,8 +443,7 @@ def skeleton_config(ctx, roles, ips, mons, cluster='ceph'):
     return conf
 
 def create_simple_monmap(ctx, remote, conf, mons,
-                         path=None,
-                         mon_bind_addrvec=False):
+                         path=None):
     """
     Writes a simple monmap based on current ceph.conf into path, or
     <testdir>/monmap by default.
@@ -469,12 +468,9 @@ def create_simple_monmap(ctx, remote, conf, mons,
         '--create',
         '--clobber',
     ]
-    for (name, addr) in addresses:
-        n = name[4:]
-        if mon_bind_addrvec:
-            args.extend(('--addv', n, addr))
-        else:
-            args.extend(('--add', n, addr))
+    for (role, addr) in addresses:
+        _, _, n = teuthology.split_role(role)
+        args.extend(('--add', n, addr))
     if not path:
         path = '{tdir}/monmap'.format(tdir=testdir)
     args.extend([