diff options
author | Ilya Dryomov <idryomov@gmail.com> | 2025-01-16 19:13:44 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2025-01-16 19:13:44 +0100 |
commit | 34b378203288f3da2cdaff19e7fae2b05d58634f (patch) | |
tree | 15cf284f914e0add2c3d087baa43f68ae8b8dff5 /qa/tasks/ceph.py | |
parent | Merge pull request #61393 from abhishek-kane/mapping_email (diff) | |
parent | qa/tasks: Include stderr on tasks badness check. (diff) | |
download | ceph-34b378203288f3da2cdaff19e7fae2b05d58634f.tar.xz ceph-34b378203288f3da2cdaff19e7fae2b05d58634f.zip |
Merge pull request #48539 from chrisphoffman/wip-57864
qa/tasks: Include stderr on tasks badness check
Reviewed-by: Adam King <adking@redhat.com>
Reviewed-by: Ilya Dryomov <idryomov@gmail.com>
Diffstat (limited to 'qa/tasks/ceph.py')
-rw-r--r-- | qa/tasks/ceph.py | 14 |
1 files changed, 12 insertions, 2 deletions
diff --git a/qa/tasks/ceph.py b/qa/tasks/ceph.py index 9b04e3dc675..8f666d2fa9b 100644 --- a/qa/tasks/ceph.py +++ b/qa/tasks/ceph.py @@ -1206,8 +1206,18 @@ def cluster(ctx, config): args.extend([ run.Raw('|'), 'head', '-n', '1', ]) - stdout = mon0_remote.sh(args) - return stdout or None + r = mon0_remote.run( + stdout=BytesIO(), + args=args, + stderr=StringIO(), + ) + stdout = r.stdout.getvalue().decode() + if stdout: + return stdout + stderr = r.stderr.getvalue() + if stderr: + return stderr + return None if first_in_ceph_log('\[ERR\]|\[WRN\]|\[SEC\]', config['log_ignorelist']) is not None: |