]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
mgr/cephadm: move NFS config/create func
authorMichael Fritch <mfritch@suse.com>
Thu, 20 Aug 2020 20:06:15 +0000 (14:06 -0600)
committerMichael Fritch <mfritch@suse.com>
Thu, 20 Aug 2020 20:06:15 +0000 (14:06 -0600)
preserve func order seen in the impl. for the other services

Signed-off-by: Michael Fritch <mfritch@suse.com>
src/pybind/mgr/cephadm/services/nfs.py

index 6824053ff20bb61afbb62256431c54f30fc39b04..bee266072e8ba9efe8dffc662dbbac82f7382fc1 100644 (file)
@@ -18,6 +18,26 @@ logger = logging.getLogger(__name__)
 class NFSService(CephadmService):
     TYPE = 'nfs'
 
+    def config(self, spec: NFSServiceSpec) -> None:
+        assert self.TYPE == spec.service_type
+        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()))
+        self.mgr.spec_store.save(spec)
+
+    def create(self, daemon_spec: CephadmDaemonSpec[NFSServiceSpec]) -> str:
+        assert self.TYPE == daemon_spec.daemon_type
+        assert daemon_spec.spec
+
+        daemon_id = daemon_spec.daemon_id
+        host = daemon_spec.host
+        spec = daemon_spec.spec
+
+        logger.info('Create daemon %s on host %s with spec %s' % (
+            daemon_id, host, spec))
+        return self.mgr._create_daemon(daemon_spec)
+
     def generate_config(self, daemon_spec: CephadmDaemonSpec) -> Tuple[Dict[str, Any], List[str]]:
         assert self.TYPE == daemon_spec.daemon_type
 
@@ -69,26 +89,6 @@ class NFSService(CephadmService):
 
         return cephadm_config, deps
 
-    def config(self, spec: NFSServiceSpec) -> None:
-        assert self.TYPE == spec.service_type
-        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()))
-        self.mgr.spec_store.save(spec)
-
-    def create(self, daemon_spec: CephadmDaemonSpec[NFSServiceSpec]) -> str:
-        assert self.TYPE == daemon_spec.daemon_type
-        assert daemon_spec.spec
-
-        daemon_id = daemon_spec.daemon_id
-        host = daemon_spec.host
-        spec = daemon_spec.spec
-
-        logger.info('Create daemon %s on host %s with spec %s' % (
-            daemon_id, host, spec))
-        return self.mgr._create_daemon(daemon_spec)
-
     def config_dashboard(self, daemon_descrs: List[DaemonDescription]):
 
         def get_set_cmd_dicts(out: str) -> List[dict]: