mgr/cephadm: fix race between check_hosts and host rm

Fixes: https://tracker.ceph.com/issues/44188
Signed-off-by: Sage Weil <sage@redhat.com>

diff --git a/src/pybind/mgr/cephadm/module.py b/src/pybind/mgr/cephadm/module.py
index f812fa7c2cb0e17ad253a933d24806768293ea13..cb478fec81c2c7e47a9b3757d29127e9c28e0eed 100644 (file)
--- a/src/pybind/mgr/cephadm/module.py
+++ b/src/pybind/mgr/cephadm/module.py
@@ -779,7 +779,10 @@ class CephadmOrchestrator(MgrModule, orchestrator.OrchestratorClientMixin):
     def _check_hosts(self):
         self.log.debug('_check_hosts')
         bad_hosts = []
-        for host, v in self.inventory.items():
+        hosts = self.inventory.keys()
+        for host in hosts:
+            if host not in self.inventory:
+                continue
             self.log.debug(' checking %s' % host)
             try:
                 out, err, code = self._run_cephadm(