From: John Spray <jspray@redhat.com>
Date: Tue, 8 Jul 2014 11:44:40 +0000 (+0100)
Subject: task: fix admin_socket for non-int ids
X-Git-Tag: 1.1.0~1315^2~2
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=785a272babca6b0bd2c4e549d7f17f55b92a2dad;p=teuthology.git

task: fix admin_socket for non-int ids

This was working for OSDs and for my config
where MDS happened to have numeric ID, but
in general service IDs are strings, not ints.

Signed-off-by: John Spray <john.spray@redhat.com>
---

diff --git a/teuthology/task/ceph_manager.py b/teuthology/task/ceph_manager.py
index 3ed9c251ff..8d638ff675 100644
--- a/teuthology/task/ceph_manager.py
+++ b/teuthology/task/ceph_manager.py
@@ -532,7 +532,7 @@ class CephManager:
         remote = None
         for _remote, roles_for_host in self.ctx.cluster.remotes.iteritems():
             for id_ in teuthology.roles_of_type(roles_for_host, service_type):
-                if int(id_) == int(service_id):
+                if id_ == str(service_id):
                     remote = _remote
         assert remote is not None
         args = [