mgr/smb: use lazy_init to create the rados store

The rados store is created when the service is deployed.

Also fix a typo identified in _lazy_init()

Signed-off-by: Sachin Prabhu <sp@spui.uk>

diff --git a/src/pybind/mgr/smb/clustermeta.py b/src/pybind/mgr/smb/clustermeta.py
index 72949c5d90579de8bc267abea76def6b32ed5644..dd412e0f3d85331780387fc3367dcefd2a3cbe46 100644 (file)
--- a/src/pybind/mgr/smb/clustermeta.py
+++ b/src/pybind/mgr/smb/clustermeta.py
@@ -179,7 +179,7 @@ _LOCK_NAME = "cluster_meta"
 def rados_object(mgr: 'MgrModule', uri: str) -> Iterator[ClusterMeta]:
     """Return a cluster meta object that will store persistent data in rados."""
     pool, ns, objname = rados_store.parse_uri(uri)
-    store = rados_store.RADOSConfigStore.init(mgr, pool)
+    store = rados_store.RADOSConfigStore.lazy_init(mgr, pool)
 
     cmeta = ClusterMeta()
     previous = {}

diff --git a/src/pybind/mgr/smb/module.py b/src/pybind/mgr/smb/module.py
index b368268c188e410ac20ae6c1b5fd0f1f268b549d..ac076b559fd28b73436ff9998ebb5a58d7aecd9b 100644 (file)
--- a/src/pybind/mgr/smb/module.py
+++ b/src/pybind/mgr/smb/module.py
@@ -71,7 +71,7 @@ class Module(orchestrator.OrchestratorClientMixin, MgrModule):
         self._priv_store = priv_store or mon_store.MonKeyConfigStore(self)
         # self._public_store = public_store or mon_store.MonKeyConfigStore(self)
         self._public_store = (
-            public_store or rados_store.RADOSConfigStore.init(self)
+            public_store or rados_store.RADOSConfigStore.lazy_init(self)
         )
         path_resolver = path_resolver or fs.CachingCephFSPathResolver(self)
         earmark_resolver = earmark_resolver or CephFSEarmarkResolver(self)

diff --git a/src/pybind/mgr/smb/rados_store.py b/src/pybind/mgr/smb/rados_store.py
index 9ac99645a65b58882fcaf88350ac1f524bd600f3..d3da535dbdb2eada03bd404d75e883e49af90adc 100644 (file)
--- a/src/pybind/mgr/smb/rados_store.py
+++ b/src/pybind/mgr/smb/rados_store.py
@@ -210,7 +210,7 @@ class RADOSConfigStore:
 
     def _lazy_init(self) -> None:
         if self._init_cb:
-            self._init_cb
+            self._init_cb()
             self._init_cb = None
 
     def __getitem__(self, key: EntryKey) -> RADOSConfigEntry: