]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
mgr/cephadm: _get_config_and_keyring: `host` is required now 36497/head
authorSebastian Wagner <sebastian.wagner@suse.com>
Fri, 7 Aug 2020 09:24:52 +0000 (11:24 +0200)
committerSebastian Wagner <sebastian.wagner@suse.com>
Fri, 7 Aug 2020 09:24:52 +0000 (11:24 +0200)
Signed-off-by: Sebastian Wagner <sebastian.wagner@suse.com>
src/pybind/mgr/cephadm/module.py
src/pybind/mgr/cephadm/services/cephadmservice.py
src/pybind/mgr/cephadm/services/nfs.py

index 8ad4a285117e847d7291603133cc90de54353a9b..11bf92d38aea4f01d95fbcac356db2fd30b6327c 100644 (file)
@@ -1815,10 +1815,10 @@ you may want to run:
                 deps.append(dd.name())
         return sorted(deps)
 
-    def _get_config_and_keyring(self, daemon_type, daemon_id, host=None,
+    def _get_config_and_keyring(self, daemon_type, daemon_id, host,
                                 keyring=None,
                                 extra_ceph_config=None):
-        # type: (str, str, Optional[str], Optional[str], Optional[str]) -> Dict[str, Any]
+        # type: (str, str, str, Optional[str], Optional[str]) -> Dict[str, Any]
         # keyring
         if not keyring:
             ename = utils.name_to_auth_entity(daemon_type, daemon_id, host=host)
index 4af2c7e35ec1ef5182fc4527ca0415e4ee70b3f0..e513e809f96c0c7bbd0fc47b0ca71260182be45d 100644 (file)
@@ -20,7 +20,7 @@ ServiceSpecs = TypeVar('ServiceSpecs', bound=ServiceSpec)
 
 class CephadmDaemonSpec(Generic[ServiceSpecs]):
     # typing.NamedTuple + Generic is broken in py36
-    def __init__(self, host, daemon_id,
+    def __init__(self, host: str, daemon_id,
                  spec: Optional[ServiceSpecs]=None,
                  network: Optional[str]=None,
                  keyring: Optional[str]=None,
@@ -34,7 +34,7 @@ class CephadmDaemonSpec(Generic[ServiceSpecs]):
 
         Would be great to have a consistent usage where all properties are set.
         """
-        self.host = host
+        self.host: str = host
         self.daemon_id = daemon_id
         daemon_type = daemon_type or (spec.service_type if spec else None)
         assert daemon_type is not None
index 354529885600570656f2b6c26eb920c5f1550222..33ce52e946bc32ce2dbe72d71db911082895fd0e 100644 (file)
@@ -64,7 +64,10 @@ class NFSService(CephadmService):
         cephadm_config.update(
             self.mgr._get_config_and_keyring(
                 daemon_type, daemon_id,
-                keyring=keyring))
+                keyring=keyring,
+                host=host
+            )
+        )
 
         return cephadm_config, deps