From b86ba640b888e2760df0d1f5482d9e3c38de7a0b Mon Sep 17 00:00:00 2001 From: Kefu Chai Date: Sun, 8 Mar 2020 13:39:59 +0800 Subject: [PATCH] qa/tasks/ceph_manager: capture stderr for COT as we are expecting the error message written to stderr, and we need to check for the error messages in it. this change addresses the regression introduced by 204ceee156cbb8a20bdf56efb0cd0610ee4c107e Fixes: https://tracker.ceph.com/issues/44500 Signed-off-by: Kefu Chai (cherry picked from commit 78308f7207a7129aaba01ea8c27e2f563d688318) --- qa/tasks/ceph_manager.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/qa/tasks/ceph_manager.py b/qa/tasks/ceph_manager.py index a5050cf2a3010..6a49bcb86689e 100644 --- a/qa/tasks/ceph_manager.py +++ b/qa/tasks/ceph_manager.py @@ -305,7 +305,7 @@ class OSDThrasher(Thrasher): if proc.exitstatus == 0: break elif (proc.exitstatus == 1 and - proc.stderr.getvalue() == "OSD has the store locked"): + six.ensure_str(proc.stderr.getvalue()) == "OSD has the store locked"): continue else: raise Exception("ceph-objectstore-tool: " @@ -417,7 +417,7 @@ class OSDThrasher(Thrasher): ]) if proc.exitstatus == 1: bogosity = "The OSD you are using is older than the exported PG" - if bogosity in proc.stderr.getvalue(): + if bogosity in six.ensure_str(proc.stderr.getvalue()): self.log("OSD older than exported PG" "...ignored") elif proc.exitstatus == 10: @@ -454,8 +454,8 @@ class OSDThrasher(Thrasher): + " --op apply-layout-settings --pool " + pool).format(id=osd) proc = imp_remote.run(args=cmd, wait=True, check_status=False, - stderr=StringIO()) - if 'Couldn\'t find pool' in proc.stderr.getvalue(): + stderr=BytesIO) + if 'Couldn\'t find pool' in six.ensure_str(proc.stderr.getvalue()): continue if proc.exitstatus: raise Exception("ceph-objectstore-tool apply-layout-settings" -- 2.39.5