[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v4 17/18] iotests: remove qemu_img_pipe_and_status()
From: |
John Snow |
Subject: |
[PATCH v4 17/18] iotests: remove qemu_img_pipe_and_status() |
Date: |
Thu, 17 Mar 2022 19:49:36 -0400 |
With the exceptional 'create' calls removed in the prior commit, change
qemu_img_log() and img_info_log() to call qemu_img() directly
instead.
For now, allow these calls to qemu-img to return non-zero on the basis
that any unusual output will be logged anyway. The very next commit
begins to enforce a successful exit code by default even for the logged
functions.
Signed-off-by: John Snow <jsnow@redhat.com>
---
tests/qemu-iotests/iotests.py | 26 +++++++-------------------
1 file changed, 7 insertions(+), 19 deletions(-)
diff --git a/tests/qemu-iotests/iotests.py b/tests/qemu-iotests/iotests.py
index 8491ff0477..c75c7470e2 100644
--- a/tests/qemu-iotests/iotests.py
+++ b/tests/qemu-iotests/iotests.py
@@ -207,15 +207,6 @@ def qemu_img_create_prepare_args(args: List[str]) ->
List[str]:
return result
-def qemu_img_pipe_and_status(*args: str) -> Tuple[str, int]:
- """
- Run qemu-img and return both its output and its exit code
- """
- is_create = bool(args and args[0] == 'create')
- full_args = qemu_img_args + qemu_img_create_prepare_args(list(args))
- return qemu_tool_pipe_and_status('qemu-img', full_args,
- drop_successful_output=is_create)
-
def qemu_img(*args: str, check: bool = True, combine_stdio: bool = True
) -> subprocess.CompletedProcess[str]:
"""
@@ -320,17 +311,14 @@ def qemu_img_info(*args: str) -> Any:
def qemu_img_map(*args: str) -> Any:
return qemu_img_json('map', "--output", "json", *args)
-def qemu_img_pipe(*args: str) -> str:
- '''Run qemu-img and return its output'''
- return qemu_img_pipe_and_status(*args)[0]
-
-def qemu_img_log(*args):
- result = qemu_img_pipe(*args)
- log(result, filters=[filter_testfiles])
+def qemu_img_log(*args: str) -> subprocess.CompletedProcess[str]:
+ result = qemu_img(*args, check=False)
+ log(result.stdout, filters=[filter_testfiles])
return result
-def img_info_log(filename, filter_path=None, use_image_opts=False,
- extra_args=()):
+def img_info_log(filename: str, filter_path: Optional[str] = None,
+ use_image_opts: bool = False, extra_args: Sequence[str] = (),
+ ) -> None:
args = ['info']
if use_image_opts:
args.append('--image-opts')
@@ -339,7 +327,7 @@ def img_info_log(filename, filter_path=None,
use_image_opts=False,
args += extra_args
args.append(filename)
- output = qemu_img_pipe(*args)
+ output = qemu_img(*args, check=False).stdout
if not filter_path:
filter_path = filename
log(filter_img_info(output, filter_path))
--
2.34.1
- [PATCH v4 08/18] iotests: add qemu_img_info(), (continued)
- [PATCH v4 08/18] iotests: add qemu_img_info(), John Snow, 2022/03/17
- [PATCH v4 07/18] iotests: use qemu_img_json() when applicable, John Snow, 2022/03/17
- [PATCH v4 10/18] iotests: add qemu_img_map() function, John Snow, 2022/03/17
- [PATCH v4 15/18] iotests: use qemu_img() in has_working_luks(), John Snow, 2022/03/17
- [PATCH v4 01/18] python/utils: add add_visual_margin() text decoration utility, John Snow, 2022/03/17
- [PATCH v4 09/18] iotests/remove-bitmap-from-backing: use qemu_img_info(), John Snow, 2022/03/17
- [PATCH v4 12/18] iotests: replace unchecked calls to qemu_img_pipe(), John Snow, 2022/03/17
- [PATCH v4 18/18] iotests: make qemu_img_log and img_info_log raise on error, John Snow, 2022/03/17
- [PATCH v4 04/18] iotests: make qemu_img raise on non-zero rc by default, John Snow, 2022/03/17
- [PATCH v4 17/18] iotests: remove qemu_img_pipe_and_status(),
John Snow <=
- [PATCH v4 16/18] iotests: replace qemu_img_log('create', ...) calls, John Snow, 2022/03/17
- [PATCH v4 11/18] iotests: change supports_quorum to use qemu_img, John Snow, 2022/03/17
- [PATCH v4 13/18] iotests/149: Remove qemu_img_pipe() call, John Snow, 2022/03/17
- [PATCH v4 14/18] iotests: remove remaining calls to qemu_img_pipe(), John Snow, 2022/03/17
- Re: [PATCH v4 00/18] iotests: add enhanced debugging info to qemu-img failures, Hanna Reitz, 2022/03/18