From: Deepika Upadhyay <dupadhya@redhat.com>
Date: Tue, 8 Dec 2020 14:34:13 +0000 (+0530)
Subject: qa/tasks/ceph_manager: use s/ByteIO/StringIO in stdout for ceph-objectstore-tool
X-Git-Tag: v14.2.17~28^2
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=ba58aa6d9e7967334fe7e2dcc97f14820bcb6fa3;p=ceph.git

qa/tasks/ceph_manager: use s/ByteIO/StringIO in stdout for ceph-objectstore-tool

wrt master, we have moved to using run_ceph_objectstore_tool which uses
StringIO for stdout and stderr, to make the changes compatible with
nautilus, replacing use of ByteIO with StringIO.

see also: d8d44ed

Signed-off-by: Deepika Upadhyay <dupadhya@redhat.com>
---

diff --git a/qa/tasks/ceph_manager.py b/qa/tasks/ceph_manager.py
index dfbd315c21df3..dfbeb4076256d 100644
--- a/qa/tasks/ceph_manager.py
+++ b/qa/tasks/ceph_manager.py
@@ -260,7 +260,7 @@ class Thrasher:
                 while proceed():
                     proc = exp_remote.run(args=cmd, wait=True,
                                           check_status=False,
-                                          stdout=BytesIO(), stderr=BytesIO())
+                                          stdout=StringIO(), stderr=StringIO())
                     if proc.exitstatus == 0:
                         break
                     elif (proc.exitstatus == 1 and
@@ -304,7 +304,7 @@ class Thrasher:
                 # If pg isn't already on this osd, then we will move it there
                 cmd = (prefix + "--op list-pgs").format(id=imp_osd)
                 proc = imp_remote.run(args=cmd, wait=True,
-                                      check_status=False, stdout=BytesIO())
+                                      check_status=False, stdout=StringIO())
                 if proc.exitstatus:
                     raise Exception("ceph-objectstore-tool: "
                                     "imp list-pgs failure with status {ret}".