[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 10/14] iotests: use qemu_img() in has_working_luks()
From: |
John Snow |
Subject: |
[PATCH 10/14] iotests: use qemu_img() in has_working_luks() |
Date: |
Tue, 8 Mar 2022 22:54:03 -0500 |
Admittedly a mostly lateral move, but qemu_img() is essentially the
replacement for qemu_img_pipe_and_status(). It will give slightly better
diagnostics on crash.
Signed-off-by: John Snow <jsnow@redhat.com>
---
tests/qemu-iotests/iotests.py | 18 +++++++++---------
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/tests/qemu-iotests/iotests.py b/tests/qemu-iotests/iotests.py
index 698d5a7fdc..1b6e28ba64 100644
--- a/tests/qemu-iotests/iotests.py
+++ b/tests/qemu-iotests/iotests.py
@@ -1450,20 +1450,20 @@ def has_working_luks() -> Tuple[bool, str]:
"""
img_file = f'{test_dir}/luks-test.luks'
- (output, status) = \
- qemu_img_pipe_and_status('create', '-f', 'luks',
- '--object', luks_default_secret_object,
- '-o', luks_default_key_secret_opt,
- '-o', 'iter-time=10',
- img_file, '1G')
+ res = qemu_img('create', '-f', 'luks',
+ '--object', luks_default_secret_object,
+ '-o', luks_default_key_secret_opt,
+ '-o', 'iter-time=10',
+ img_file, '1G',
+ check=False)
try:
os.remove(img_file)
except OSError:
pass
- if status != 0:
- reason = output
- for line in output.splitlines():
+ if res.returncode:
+ reason = res.stdout
+ for line in res.stdout.splitlines():
if img_file + ':' in line:
reason = line.split(img_file + ':', 1)[1].strip()
break
--
2.34.1
- Re: [PATCH 04/14] iotests/remove-bitmap-from-backing: use qemu_img_info(), (continued)
- [PATCH 03/14] iotests: add qemu_img_info(), John Snow, 2022/03/08
- [PATCH 05/14] iotests: add qemu_img_map() function, John Snow, 2022/03/08
- [PATCH 06/14] iotests: change supports_quorum to use qemu_img, John Snow, 2022/03/08
- [PATCH 07/14] iotests: replace unchecked calls to qemu_img_pipe(), John Snow, 2022/03/08
- [PATCH 10/14] iotests: use qemu_img() in has_working_luks(),
John Snow <=
- [PATCH 08/14] iotests/149: Remove qemu_img_pipe() call, John Snow, 2022/03/08
- [PATCH 12/14] iotests: remove qemu_img_pipe_and_status(), John Snow, 2022/03/08
[PATCH 11/14] iotests: replace qemu_img_log('create', ...) calls, John Snow, 2022/03/08