From 69d82b9e129ceb179dc33782b7969e9419cb4987 Mon Sep 17 00:00:00 2001 From: Laura Flores Date: Wed, 24 Aug 2022 22:23:45 +0000 Subject: [PATCH] src/pybind/mgr/telemetry: parse `outb` instead of `outs` Following the merge of https://github.com/ceph/ceph/pull/47650, which fixes the confusion between std out and std err in admin socket commands, we will need to reference the out stream (outb) instead of the error stream (outs) when we parse heap stats. Signed-off-by: Laura Flores (cherry picked from commit 025f8105a59db22ddc28abab288a43738a9967e4) --- src/pybind/mgr/telemetry/module.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/pybind/mgr/telemetry/module.py b/src/pybind/mgr/telemetry/module.py index 1b44fc0d26458..779cd529b32a7 100644 --- a/src/pybind/mgr/telemetry/module.py +++ b/src/pybind/mgr/telemetry/module.py @@ -521,8 +521,8 @@ class Module(MgrModule): if r != 0: self.log.error("Invalid command dictionary: {}".format(cmd_dict)) else: - if 'tcmalloc heap stats' in outs: - values = [int(i) for i in outs.split() if i.isdigit()] + if 'tcmalloc heap stats' in outb: + values = [int(i) for i in outb.split() if i.isdigit()] # `categories` must be ordered this way for the correct output to be parsed categories = ['use_by_application', 'page_heap_freelist', -- 2.39.5