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 <js...@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 <ebl...@redhat.com> -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3266 Virtualization: qemu.org | libvirt.org
