[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v5 18/18] iotests: make qemu_img_log and img_info_log raise o
From: |
Eric Blake |
Subject: |
Re: [PATCH v5 18/18] iotests: make qemu_img_log and img_info_log raise on error |
Date: |
Mon, 21 Mar 2022 16:33:03 -0500 |
User-agent: |
NeoMutt/20211029-454-6adf99 |
On Mon, Mar 21, 2022 at 04:16:18PM -0400, John Snow wrote:
> Add a `check: bool = True` parameter to both functions and make their
> qemu_img() invocations raise on error by default.
>
> users of img_info_log:
> 206, 207, 210, 211, 212, 213, 237, 242, 266, 274, 302
>
> users of qemu_img_log:
> 044, 209, 274, 302, 304
>
> iotests 242 and 266 need to use check=False for their negative tests.
> iotests 206, 210, 211, 212, 213, 237, 274 and 302 continue working
> normally.
>
> As of this commit, all calls to QEMU_IMG made from iotests enforce a
> return code of zero by default unless explicitly disabled or suppressed
> by passing check=False or with an exception handler.
>
> Signed-off-by: John Snow <jsnow@redhat.com>
> ---
> tests/qemu-iotests/242 | 2 +-
> tests/qemu-iotests/266 | 2 +-
> tests/qemu-iotests/iotests.py | 8 +++++---
> 3 files changed, 7 insertions(+), 5 deletions(-)
Reviewed-by: Eric Blake <eblake@redhat.com>
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
- [PATCH v5 03/18] iotests: Remove explicit checks for qemu_img() == 0, (continued)
- [PATCH v5 03/18] iotests: Remove explicit checks for qemu_img() == 0, John Snow, 2022/03/21
- [PATCH v5 11/18] iotests: change supports_quorum to use qemu_img, John Snow, 2022/03/21
- [PATCH v5 08/18] iotests: add qemu_img_info(), John Snow, 2022/03/21
- [PATCH v5 12/18] iotests: replace unchecked calls to qemu_img_pipe(), John Snow, 2022/03/21
- [PATCH v5 13/18] iotests/149: Remove qemu_img_pipe() call, John Snow, 2022/03/21
- [PATCH v5 10/18] iotests: add qemu_img_map() function, John Snow, 2022/03/21
- [PATCH v5 15/18] iotests: use qemu_img() in has_working_luks(), John Snow, 2022/03/21
- [PATCH v5 16/18] iotests: replace qemu_img_log('create', ...) calls, John Snow, 2022/03/21
- [PATCH v5 14/18] iotests: remove remaining calls to qemu_img_pipe(), John Snow, 2022/03/21
- [PATCH v5 18/18] iotests: make qemu_img_log and img_info_log raise on error, John Snow, 2022/03/21
- Re: [PATCH v5 18/18] iotests: make qemu_img_log and img_info_log raise on error,
Eric Blake <=
- [PATCH v5 17/18] iotests: remove qemu_img_pipe_and_status(), John Snow, 2022/03/21
- Re: [PATCH v5 00/18] iotests: add enhanced debugging info to qemu-img failures, Hanna Reitz, 2022/03/22