[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [PATCH 1/2] BootLinuxConsoleTest: Do not log empty lines
From: |
Cleber Rosa |
Subject: |
Re: [Qemu-arm] [PATCH 1/2] BootLinuxConsoleTest: Do not log empty lines |
Date: |
Wed, 5 Jun 2019 11:57:17 -0400 |
User-agent: |
Mutt/1.11.4 (2019-03-13) |
On Tue, May 21, 2019 at 12:06:34AM +0200, Philippe Mathieu-Daudé wrote:
> Avoid to log empty lines in console debug logs.
>
> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> tests/acceptance/boot_linux_console.py | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/tests/acceptance/boot_linux_console.py
> b/tests/acceptance/boot_linux_console.py
> index d5c500ea30..f593f3858e 100644
> --- a/tests/acceptance/boot_linux_console.py
> +++ b/tests/acceptance/boot_linux_console.py
> @@ -37,8 +37,10 @@ class BootLinuxConsole(Test):
> console = self.vm.console_socket.makefile()
> console_logger = logging.getLogger('console')
> while True:
> - msg = console.readline()
> - console_logger.debug(msg.strip())
> + msg = console.readline().strip()
> + if not msg:
> + continue
> + console_logger.debug(msg)
> if success_message in msg:
> break
> if failure_message in msg:
> --
> 2.19.1
>
This is particularly useful in the
boot_linux_console.py:BootLinuxConsole.test_s390x_s390_ccw_virtio test, where
in my experience empty lines were being logged.
Reviewed-by: Cleber Rosa <address@hidden>
Tested-by: Cleber Rosa <address@hidden>
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-arm] [PATCH 1/2] BootLinuxConsoleTest: Do not log empty lines,
Cleber Rosa <=