Merge pull request ceph#59681 from adk3798/cephadm-enable-module-boot…

…strap-retry cephadm: bootstrap: verify orch module is running before setting cephadm backend Reviewed-by: John Mulligan <[email protected]>
linuxbox2 · Sep 11, 2024 · 0537c19 · 0537c19
2 parents 8b0f3a5 + fcdae78
commit 0537c19
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/src/cephadm/cephadm.py b/src/cephadm/cephadm.py
@@ -2410,6 +2410,12 @@ def enable_cephadm_mgr_module(
     logger.info('Enabling cephadm module...')
     cli(['mgr', 'module', 'enable', 'cephadm'])
     wait_for_mgr_restart()
+    # https://tracker.ceph.com/issues/67969
+    # luckily `ceph mgr module enable <module>` returns
+    # a zero rc when the module is already enabled so
+    # this is no issue even if it is unnecessary
+    logger.info('Verifying orchestrator module is enabled...')
+    cli(['mgr', 'module', 'enable', 'orchestrator'])
     logger.info('Setting orchestrator backend to cephadm...')
     cli(['orch', 'set', 'backend', 'cephadm'])