mgr/cephadm: clean-up service module type checking

- add return type to `config` func
- add missing `NFSServiceSpec` annotation to `config` func

Signed-off-by: Michael Fritch <mfritch@suse.com>

diff --git a/src/pybind/mgr/cephadm/module.py b/src/pybind/mgr/cephadm/module.py
index 1b3d18b8292e6fe8de61f23c2c0b9228ab1f8e88..5605f5e23193b672a3918bba9567892c38fca850 100644 (file)
--- a/src/pybind/mgr/cephadm/module.py
+++ b/src/pybind/mgr/cephadm/module.py
@@ -1799,7 +1799,7 @@ you may want to run:
             'rgw': self.rgw_service.config,
             'nfs': self.nfs_service.config,
             'iscsi': self.iscsi_service.config,
-        }.get(service_type)
+        }.get(service_type)  # type: ignore
 
     def _apply_service(self, spec: ServiceSpec) -> bool:
         """

diff --git a/src/pybind/mgr/cephadm/services/cephadmservice.py b/src/pybind/mgr/cephadm/services/cephadmservice.py
index e53de8189251a84a15ff7eb8f483f7a0ab693c6f..64e67740b8f96e9cd44041595f8c8f05a2421c94 100644 (file)
--- a/src/pybind/mgr/cephadm/services/cephadmservice.py
+++ b/src/pybind/mgr/cephadm/services/cephadmservice.py
@@ -319,7 +319,7 @@ class MgrService(CephadmService):
 class MdsService(CephadmService):
     TYPE = 'mds'
 
-    def config(self, spec: ServiceSpec):
+    def config(self, spec: ServiceSpec) -> None:
         # ensure mds_join_fs is set for these daemons
         assert spec.service_id
         ret, out, err = self.mgr.check_mon_command({
@@ -348,7 +348,7 @@ class MdsService(CephadmService):
 class RgwService(CephadmService):
     TYPE = 'rgw'
 
-    def config(self, spec: RGWSpec):
+    def config(self, spec: RGWSpec) -> None:
         # ensure rgw_realm and rgw_zone is set for these daemons
         ret, out, err = self.mgr.check_mon_command({
             'prefix': 'config set',

diff --git a/src/pybind/mgr/cephadm/services/iscsi.py b/src/pybind/mgr/cephadm/services/iscsi.py
index 127783c13e4409a8a50126a99ecabd22ec14a027..1c46dd674dea9c4bd48f369eaa8efea3fd5fedd7 100644 (file)
--- a/src/pybind/mgr/cephadm/services/iscsi.py
+++ b/src/pybind/mgr/cephadm/services/iscsi.py
@@ -15,7 +15,7 @@ logger = logging.getLogger(__name__)
 class IscsiService(CephadmService):
     TYPE = 'iscsi'
 
-    def config(self, spec: IscsiServiceSpec):
+    def config(self, spec: IscsiServiceSpec) -> None:
         self.mgr._check_pool_exists(spec.pool, spec.service_name())
 
         logger.info('Saving service %s spec with placement %s' % (

diff --git a/src/pybind/mgr/cephadm/services/nfs.py b/src/pybind/mgr/cephadm/services/nfs.py
index 54d3e244f2a6cb2f1a75c8ccaf0c6cf546c41e4f..a7b3060c5fb74177b5990c328ce8b42fad948e75 100644 (file)
--- a/src/pybind/mgr/cephadm/services/nfs.py
+++ b/src/pybind/mgr/cephadm/services/nfs.py
@@ -66,7 +66,7 @@ class NFSService(CephadmService):
 
         return cephadm_config, deps
 
-    def config(self, spec):
+    def config(self, spec: NFSServiceSpec) -> None:
         self.mgr._check_pool_exists(spec.pool, spec.service_name())
         logger.info('Saving service %s spec with placement %s' % (
             spec.service_name(), spec.placement.pretty_str()))