[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] qemu-iotests: Use -nographic in 182
From: |
Kashyap Chamarthy |
Subject: |
Re: [Qemu-devel] [PATCH] qemu-iotests: Use -nographic in 182 |
Date: |
Mon, 30 Oct 2017 09:39:16 +0100 |
User-agent: |
Mutt/1.6.0.1 (2016-04-01) |
On Sun, Oct 29, 2017 at 01:53:38PM +0100, Kevin Wolf wrote:
> This avoids that random UI frontend error messages end up in the output.
> In particular, we were seeing this line in CI error logs:
>
> +Unable to init server: Could not connect: Connection refused
>
> Signed-off-by: Kevin Wolf <address@hidden>
> ---
> tests/qemu-iotests/182 | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/qemu-iotests/182 b/tests/qemu-iotests/182
> index 2e078ceed8..4b31592fb8 100755
> --- a/tests/qemu-iotests/182
> +++ b/tests/qemu-iotests/182
> @@ -62,7 +62,7 @@ _launch_qemu -drive
> file=$TEST_IMG,if=none,id=drive0,file.locking=on \
>
> echo
> echo "Starting a second QEMU using the same image should fail"
> -echo 'quit' | $QEMU -monitor stdio \
> +echo 'quit' | $QEMU -nographic -monitor stdio \
Isn't "-display none" preferred instead of "-nographic"?
Assuming you'll change it to "-display none":
Reviewed-by: Kashyap CHamarthy <address@hidden>
[...]
--
/kashyap