[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v5 08/12] tests/acceptance: wait() instead of shutdown() wher
From: |
Cleber Rosa |
Subject: |
Re: [PATCH v5 08/12] tests/acceptance: wait() instead of shutdown() where appropriate |
Date: |
Mon, 13 Jul 2020 23:37:31 -0400 |
On Fri, Jul 10, 2020 at 01:06:45AM -0400, John Snow wrote:
> When issuing 'reboot' to a VM with the no-reboot option, that VM will
> exit. When then issuing a shutdown command, the cleanup may race.
>
> Add calls to vm.wait() which will gracefully mark the VM as having
> exited. Subsequent vm.shutdown() calls in generic tearDown code will not
> race when called after completion of the call.
>
> Signed-off-by: John Snow <jsnow@redhat.com>
> ---
> tests/acceptance/boot_linux_console.py | 10 ++++++++++
> tests/acceptance/linux_ssh_mips_malta.py | 2 ++
> 2 files changed, 12 insertions(+)
>
Reviewed-by: Cleber Rosa <crosa@redhat.com>
Tested-by: Cleber Rosa <crosa@redhat.com>
signature.asc
Description: PGP signature
- Re: [PATCH v5 04/12] python/machine.py: Perform early cleanup for wait() calls, too, (continued)
- [PATCH v5 06/12] python/machine.py: Add a configurable timeout to shutdown(), John Snow, 2020/07/10
- [PATCH v5 07/12] python/machine.py: Make wait() call shutdown(), John Snow, 2020/07/10
- [PATCH v5 08/12] tests/acceptance: wait() instead of shutdown() where appropriate, John Snow, 2020/07/10
- [PATCH v5 09/12] tests/acceptance: Don't test reboot on cubieboard, John Snow, 2020/07/10
- Re: [PATCH v5 09/12] tests/acceptance: Don't test reboot on cubieboard, Cleber Rosa, 2020/07/13
- [PATCH v5 10/12] python/machine.py: split shutdown into hard and soft flavors, John Snow, 2020/07/10