]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
mgr/cephadm: fix removing offline hosts with ingress daemons 49827/head
authorAdam King <adking@redhat.com>
Sun, 22 Jan 2023 18:59:51 +0000 (13:59 -0500)
committerAdam King <adking@redhat.com>
Sun, 22 Jan 2023 18:59:51 +0000 (13:59 -0500)
We need to convert the daemon type to service type here
or we'll try to access the "haproxy" or "keepalived" index
of the cephadm_services dictionary, which doesn't have those
keys (but does have an "ingress" key)

Fixes: https://tracker.ceph.com/issues/58537
Signed-off-by: Adam King <adking@redhat.com>
src/pybind/mgr/cephadm/module.py

index b86a00d6fcad07b4bbf78506000052ed7db959ac..c999c91e32a0cabd82fdcd7618383274477542f6 100644 (file)
@@ -1571,8 +1571,8 @@ Then run the following:
                 self.log.info(f"removing: {d.name()}")
 
                 if d.daemon_type != 'osd':
-                    self.cephadm_services[str(d.daemon_type)].pre_remove(d)
-                    self.cephadm_services[str(d.daemon_type)].post_remove(d, is_failed_deploy=False)
+                    self.cephadm_services[daemon_type_to_service(str(d.daemon_type))].pre_remove(d)
+                    self.cephadm_services[daemon_type_to_service(str(d.daemon_type))].post_remove(d, is_failed_deploy=False)
                 else:
                     cmd_args = {
                         'prefix': 'osd purge-actual',