[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] tests/avocado: configure acpi-bits to use avocado timeout
From: |
Alex Bennée |
Subject: |
Re: [PATCH] tests/avocado: configure acpi-bits to use avocado timeout |
Date: |
Tue, 15 Nov 2022 22:47:33 +0000 |
User-agent: |
mu4e 1.9.2; emacs 28.2.50 |
John Snow <jsnow@redhat.com> writes:
> Instead of using a hardcoded timeout, just rely on Avocado's built-in
> test case timeout. This helps avoid timeout issues on machines where 60
> seconds is not sufficient.
>
> Signed-off-by: John Snow <jsnow@redhat.com>
> ---
> tests/avocado/acpi-bits.py | 10 ++--------
> 1 file changed, 2 insertions(+), 8 deletions(-)
>
> diff --git a/tests/avocado/acpi-bits.py b/tests/avocado/acpi-bits.py
> index 8745a58a766..ac13e22dc93 100644
> --- a/tests/avocado/acpi-bits.py
> +++ b/tests/avocado/acpi-bits.py
> @@ -385,12 +385,6 @@ def test_acpi_smbios_bits(self):
> self._vm.launch()
> # biosbits has been configured to run all the specified test suites
> # in batch mode and then automatically initiate a vm shutdown.
> - # sleep for maximum of one minute
> - max_sleep_time = time.monotonic() + 60
> - while self._vm.is_running() and time.monotonic() < max_sleep_time:
> - time.sleep(1)
> -
> - self.assertFalse(time.monotonic() > max_sleep_time,
> - 'The VM seems to have failed to shutdown in time')
> -
We might want some wait for consoles as well depending on what is output
during the run.
> + # Rely on avocado's unit test timeout.
> + self._vm.wait(timeout=None)
> self.parse_log()
--
Alex Bennée
Re: [PATCH] tests/avocado: configure acpi-bits to use avocado timeout, Ani Sinha, 2022/11/15
Re: [PATCH] tests/avocado: configure acpi-bits to use avocado timeout, Ani Sinha, 2022/11/17