From: John Spray <john.spray@redhat.com>
Date: Fri, 18 May 2018 15:44:34 +0000 (+0100)
Subject: mgr/status: output to stdout, not stderr
X-Git-Tag: v13.2.0~30^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F22135%2Fhead;p=ceph.git

mgr/status: output to stdout, not stderr

I think this was just me forgetting the order
of the arguments when writing the return statements.

Fixes: http://tracker.ceph.com/issues/24175
Signed-off-by: John Spray <john.spray@redhat.com>
(cherry picked from commit c18db575ca87906faa201c4d3bde75b99148d18f)
---

diff --git a/src/pybind/mgr/status/module.py b/src/pybind/mgr/status/module.py
index 5175625702563..b4a0f25e0af70 100644
--- a/src/pybind/mgr/status/module.py
+++ b/src/pybind/mgr/status/module.py
@@ -249,7 +249,7 @@ class Module(MgrModule):
                 ])
             output += version_table.get_string() + "\n"
 
-        return 0, "", output
+        return 0, output, ""
 
     def handle_osd_status(self, cmd):
         osd_table = PrettyTable(['id', 'host', 'used', 'avail', 'wr ops', 'wr data', 'rd ops', 'rd data', 'state'])
@@ -301,7 +301,7 @@ class Module(MgrModule):
                                ','.join(osd['state']),
                                ])
 
-        return 0, "", osd_table.get_string()
+        return 0, osd_table.get_string(), ""
 
     def handle_command(self, cmd):
         self.log.error("handle_command")