[v8] qemu_iotests: improve debugging options

[PATCH v8 16/16] docs/devel/testing: add -p option to the debug section of QEMU iotests

Posted by Emanuele Giuseppe Esposito 4 years, 7 months ago

Signed-off-by: Emanuele Giuseppe Esposito <eesposit@redhat.com>
Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
---
 docs/devel/testing.rst | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/docs/devel/testing.rst b/docs/devel/testing.rst
index 719accdb1e..e5311cb167 100644
--- a/docs/devel/testing.rst
+++ b/docs/devel/testing.rst
@@ -249,6 +249,10 @@ a failing test:
 * ``-d`` (debug) just increases the logging verbosity, showing
   for example the QMP commands and answers.
 
+* ``-p`` (print) redirect QEMU’s stdout and stderr to the test output,
+  instead of saving it into a log file in
+  ``$TEST_DIR/qemu-machine-<random_string>``.
+
 Test case groups
 ----------------
 
-- 
2.31.1

Re: [PATCH v8 16/16] docs/devel/testing: add -p option to the debug section of QEMU iotests

Posted by Max Reitz 4 years, 6 months ago

On 05.07.21 08:57, Emanuele Giuseppe Esposito wrote:
> Signed-off-by: Emanuele Giuseppe Esposito <eesposit@redhat.com>
> Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
> ---
>   docs/devel/testing.rst | 4 ++++
>   1 file changed, 4 insertions(+)
>
> diff --git a/docs/devel/testing.rst b/docs/devel/testing.rst
> index 719accdb1e..e5311cb167 100644
> --- a/docs/devel/testing.rst
> +++ b/docs/devel/testing.rst
> @@ -249,6 +249,10 @@ a failing test:
>   * ``-d`` (debug) just increases the logging verbosity, showing
>     for example the QMP commands and answers.
>   
> +* ``-p`` (print) redirect QEMU’s stdout and stderr to the test output,

Sorry, my bad: s/redirect/redirects/

With that fixed:

Reviewed-by: Max Reitz <mreitz@redhat.com>

Max

> +  instead of saving it into a log file in
> +  ``$TEST_DIR/qemu-machine-<random_string>``.
> +
>   Test case groups
>   ----------------
>