]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
mgr/prometheus: Fix KeyError in get_mgr_status 30421/head
authorSebastian Wagner <sebastian.wagner@suse.com>
Tue, 17 Sep 2019 07:17:08 +0000 (09:17 +0200)
committerSebastian Wagner <sebastian.wagner@suse.com>
Tue, 17 Sep 2019 12:00:42 +0000 (14:00 +0200)
`ceph_release` is not necessarily a valid key for `mgr_map['always_on_modules']`

Fixes: https://tracker.ceph.com/issues/41878
Signed-off-by: Sebastian Wagner <sebastian.wagner@suse.com>
src/pybind/mgr/prometheus/module.py

index 1df5d6b6d63e8c59f91f89893be6d149415c3803..9aa7ba6421597873efa43dbe4d2f00250f97787c 100644 (file)
@@ -476,6 +476,7 @@ class Module(MgrModule):
 
         all_modules = {module.get('name'):module.get('can_run') for module in mgr_map['available_modules']}
 
+        ceph_release = None
         for mgr in all_mgrs:
             host_version = servers.get((mgr, 'mgr'), ('', ''))
             if mgr == active:
@@ -491,7 +492,7 @@ class Module(MgrModule):
             self.metrics['mgr_status'].set(_state, (
                 'mgr.{}'.format(mgr), 
             ))
-        always_on_modules = mgr_map['always_on_modules'][ceph_release]
+        always_on_modules = mgr_map['always_on_modules'].get(ceph_release, [])
         active_modules = list(always_on_modules)
         active_modules.extend(mgr_map['modules'])