From: Konstantin Shalygin Date: Thu, 21 Feb 2019 07:22:21 +0000 (+0700) Subject: mgr/prometheus: get osd_objectstore once instead twice. X-Git-Tag: v14.1.1~131^2 X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=bf3231ba17c7eaada8a24e8cd8a57b8ec6216dc4;p=ceph.git mgr/prometheus: get osd_objectstore once instead twice. After merging #25234 & #24821 separetly we get in get_metadata_and_osd_status() double osd_metadata.get('osd_objectstore') call. Signed-off-by: Konstantin Shalygin --- diff --git a/src/pybind/mgr/prometheus/module.py b/src/pybind/mgr/prometheus/module.py index ab92bee15112..fc5e3e95c357 100644 --- a/src/pybind/mgr/prometheus/module.py +++ b/src/pybind/mgr/prometheus/module.py @@ -498,15 +498,14 @@ class Module(MgrModule): 'osd.{}'.format(id_), )) - osd_objectstore = osd_metadata.get('osd_objectstore', None) - if osd_objectstore == "filestore": + if obj_store == "filestore": # collect filestore backend device osd_dev_node = osd_metadata.get( 'backend_filestore_dev_node', None) # collect filestore journal device osd_wal_dev_node = osd_metadata.get('osd_journal', '') osd_db_dev_node = '' - elif osd_objectstore == "bluestore": + elif obj_store == "bluestore": # collect bluestore backend device osd_dev_node = osd_metadata.get( 'bluestore_bdev_dev_node', None)