On 7/14/20 12:20 AM, Cleber Rosa wrote:
> On Fri, Jul 10, 2020 at 01:06:49AM -0400, John Snow wrote:
>> Machine.wait() does not appear to be used except in the acceptance tests,
>> and an infinite timeout by default in a test suite is not the most helpful.
>>
>> Change it to 3 seconds, like the default shutdown timeout.
>>
>> Signed-off-by: John Snow <js...@redhat.com>
>> ---
>> python/qemu/machine.py | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>
> Well, for the acceptance tests, there's usually a test wide timeout,
> but this is indeed a good idea!
>
> Reviewed-by: Cleber Rosa <cr...@redhat.com>
> Tested-by: Cleber Rosa <cr...@redhat.com>
>
Yes, there's a bigger timeout for acceptance tests, but iotests doesn't
have the same just yet.
In general, it helps for most of the python library methods to time out
by default to prevent hangs in the various test suites.
So, anticipating that iotest callers will probably want to use wait()
sooner or later, I just went ahead and made the change primarily for
consistency again.
--js
