Re: [Qemu-devel] [PATCH] iotest: fix python based IO tests

[Max Reitz <mreitz@redhat.com>]

[-- Attachment #1: Type: text/plain, Size: 1728 bytes --]

On 26.07.2016 15:13, Daniel P. Berrange wrote:
> The previous commit refactoring iotests.py:
> 
>   commit 66613974468fb6e1609fb3eabf55981b1ee436cf
>   Author: Daniel P. Berrange <berrange@redhat.com>
>   Date:   Wed Jul 20 14:23:10 2016 +0100
> 
>     scripts: refactor the VM class in iotests for reuse
> 
> was not properly tested and included a number of broken
> bits.
> 
>  - The 'event_match' method was not moved into qemu.py
>  - The 'self._args' list parameter in QEMUMachine needs
>    to be copied otherwise modifications will affect the
>    global 'qemu_opts' variable in iotests.py
>  - The QEMUQtestMachine class methods had inverted
>    parameter order for the super() calls
>  - The QEMUQtestMachine class forgot to add
>    '-machine accel=qtest'
>  - The QEMUQtestMachine class constructor needs to set
>    a default 'name' value before using it as it may
>    be None
>  - The QEMUQtestMachine class constructor needs to use
>    named parameters when calling the super constructor
>    as it is leaving out some positional parameters.
>  - The 'qemu_prog' variable should be a string not a
>    list in iotests.py
>  - The VM classs constructor needs to use named
>    parameters when calling the super constructor
>    as it is leaving out some positional parameters.
> 
> Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
> ---
>  scripts/qemu.py               | 19 ++++++++++++++++++-
>  scripts/qtest.py              | 15 +++++++++------
>  tests/qemu-iotests/iotests.py | 23 +++--------------------
>  3 files changed, 30 insertions(+), 27 deletions(-)

Thanks! :-)

Applied to my block tree:

https://github.com/XanClic/qemu/commits/block

Max


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 498 bytes --]