Re: [Qemu-block] [PATCH v3 0/2] qemu-iotests: fix cleanup of background processes

[Max Reitz]

On 02.11.2015 21:31, Max Reitz wrote:
> On 30.10.2015 20:25, Jeff Cody wrote:
>> Changes from v2:
>>
>> * Pulled patch 2 into this series
>>
>> * Patch 1: Moved non-empty test conditionals inside
>>            pid file existance check (thanks Max)
>>
>>            Added a fix in for patch 058, for its
>>            self-launched qemu-nbd instance (thanks Max)
>>
>> * Patch 2: Only print the valgrind logs if the exit error
>>            matches (thanks Max) 
>>
>> Changes from v1:
>>
>> * use 'read' instead of 'cat' (thanks Eric)
>> * quote variable in variable test (thanks Eric)
>>
>> Jeff Cody (2):
>>   qemu-iotests: fix cleanup of background processes
>>   qemu-iotests: fix -valgrind option for check
>>
>>  tests/qemu-iotests/039.out       | 30 +++++++++++++++++++++++++-----
>>  tests/qemu-iotests/058           | 12 ++++++++----
>>  tests/qemu-iotests/061.out       | 12 ++++++++++--
>>  tests/qemu-iotests/137.out       |  6 +++++-
>>  tests/qemu-iotests/common        |  9 ++-------
>>  tests/qemu-iotests/common.config | 32 +++++++++++++++++++++++++++++---
>>  tests/qemu-iotests/common.qemu   | 18 ++++++++++++------
>>  tests/qemu-iotests/common.rc     | 18 +++++-------------
>>  8 files changed, 96 insertions(+), 41 deletions(-)
> 
> Thanks, applied to my block tree with the three occurrences of ! -n in
> patch 1 changed to -z:

Err, of course it's the other way around, ! -z to -n.

Max

signature.asc
Description: OpenPGP digital signature