From: Michael Fritch <mfritch@suse.com>
Date: Tue, 1 Sep 2020 19:32:44 +0000 (-0600)
Subject: mgr/cephadm: refactor `get_auth_entity()`
X-Git-Tag: v15.2.9~122^2~44^2~40
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=45cf6732c38b211087f4d0c35801abbaef198ac9;p=ceph.git

mgr/cephadm: refactor `get_auth_entity()`

- convert to f-strings
- mgr keyring format is same as osd, mds
- crash client keyring is same as nfs, iscsi, etc

Signed-off-by: Michael Fritch <mfritch@suse.com>
(cherry picked from commit 9d2031f01de4542f9a03f1234d5794395ae9162b)
---

diff --git a/src/pybind/mgr/cephadm/services/cephadmservice.py b/src/pybind/mgr/cephadm/services/cephadmservice.py
index f43c61f88bd..476dde93751 100644
--- a/src/pybind/mgr/cephadm/services/cephadmservice.py
+++ b/src/pybind/mgr/cephadm/services/cephadmservice.py
@@ -234,17 +234,15 @@ class CephadmService(metaclass=ABCMeta):
         Map the daemon id to a cephx keyring entity name
         """
         if self.TYPE in ['rgw', 'rbd-mirror', 'nfs', "iscsi"]:
-            return AuthEntity('client.' + self.TYPE + "." + daemon_id)
+            return AuthEntity(f'client.{self.TYPE}.{daemon_id}')
         elif self.TYPE == 'crash':
             if host == "":
                 raise OrchestratorError("Host not provided to generate <crash> auth entity name")
-            return AuthEntity('client.' + self.TYPE + "." + host)
+            return AuthEntity(f'client.{self.TYPE}.{host}')
         elif self.TYPE == 'mon':
             return AuthEntity('mon.')
-        elif self.TYPE == 'mgr':
-            return AuthEntity(self.TYPE + "." + daemon_id)
-        elif self.TYPE in ['osd', 'mds', 'client']:
-            return AuthEntity(self.TYPE + "." + daemon_id)
+        elif self.TYPE in ['mgr', 'osd', 'mds']:
+            return AuthEntity(f'{self.TYPE}.{daemon_id}')
         else:
             raise OrchestratorError("unknown daemon type")