From 056bc08d51306f4e4d031d1e3ff68ad7312c55d8 Mon Sep 17 00:00:00 2001 From: Boris Ranto Date: Tue, 1 May 2018 01:12:02 +0200 Subject: [PATCH] prometheus: Handle the TIME perf counter type metrics This patch correctly sets the PERFCOUNTER_MASK to 3 so that the PERFCOUNTER_TIME metrics are not ignored by the mgr_module code. It also converts the TIME metrics from nanoseconds to seconds just like the ceph perf dump does and exposes the metrics via prometheus module. Signed-off-by: Boris Ranto --- src/pybind/mgr/mgr_module.py | 9 ++++++++- src/pybind/mgr/prometheus/module.py | 3 ++- 2 files changed, 10 insertions(+), 2 deletions(-) diff --git a/src/pybind/mgr/mgr_module.py b/src/pybind/mgr/mgr_module.py index b51952ebd18e..ea79771fbabe 100644 --- a/src/pybind/mgr/mgr_module.py +++ b/src/pybind/mgr/mgr_module.py @@ -229,7 +229,7 @@ class MgrModule(ceph_module.BaseMgrModule): PERFCOUNTER_LONGRUNAVG = 4 PERFCOUNTER_COUNTER = 8 PERFCOUNTER_HISTOGRAM = 0x10 - PERFCOUNTER_TYPE_MASK = ~2 + PERFCOUNTER_TYPE_MASK = ~3 # units supported BYTES = 0 @@ -337,6 +337,13 @@ class MgrModule(ceph_module.BaseMgrModule): return '' + def _perfvalue_to_value(self, stattype, value): + if stattype & self.PERFCOUNTER_TIME: + # Convert from ns to seconds + return value / 1000000000.0 + else: + return value + def _unit_to_str(self, unit): if unit == self.NONE: return "/s" diff --git a/src/pybind/mgr/prometheus/module.py b/src/pybind/mgr/prometheus/module.py index c7daa128dd8a..dea9b5971788 100644 --- a/src/pybind/mgr/prometheus/module.py +++ b/src/pybind/mgr/prometheus/module.py @@ -554,7 +554,8 @@ class Module(MgrModule): ("ceph_daemon",), )) - self.metrics.append(path, counter_info['value'], (daemon,)) + value = self._perfvalue_to_value(counter_info['type'], counter_info['value']) + self.metrics.append(path, value, (daemon,)) # It is sufficient to reset the pending metrics once per scrape self.metrics.reset() -- 2.47.3