]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
src/pybind/mgr/telemetry: parse `outb` instead of `outs` 47826/head
authorLaura Flores <lflores@redhat.com>
Wed, 24 Aug 2022 22:23:45 +0000 (22:23 +0000)
committerLaura Flores <lflores@redhat.com>
Fri, 26 Aug 2022 22:32:50 +0000 (17:32 -0500)
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 <lflores@redhat.com>
(cherry picked from commit 025f8105a59db22ddc28abab288a43738a9967e4)

src/pybind/mgr/telemetry/module.py

index 1b44fc0d264589f206b76d8929ec42dbca89f74a..779cd529b32a749a870236ad14272c9c4941ba5b 100644 (file)
@@ -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',