[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH RESEND 49/50] iotests: More options for VM.add_d
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH RESEND 49/50] iotests: More options for VM.add_drive() |
Date: |
Wed, 28 Jan 2015 14:27:21 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.4.0 |
On 01/27/2015 12:46 PM, Max Reitz wrote:
> This patch allows specifying the interface to be used for the drive, and
> makes specifying a path optional (if the path is None, the "file" option
> will be omitted, thus creating an empty drive).
>
> Signed-off-by: Max Reitz <address@hidden>
> ---
> tests/qemu-iotests/iotests.py | 9 ++++++---
> 1 file changed, 6 insertions(+), 3 deletions(-)
>
> +
> + if not path is None:
> + options.append('file=%s' % path)
Works, but it is more idiomatic to use 'if path is not None:', based on
a grep of the source tree.
With that minor switch,
Reviewed-by: Eric Blake <address@hidden>
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH RESEND 42/50] blockdev: Implement eject with basic operations, (continued)
- [Qemu-devel] [PATCH RESEND 42/50] blockdev: Implement eject with basic operations, Max Reitz, 2015/01/27
- [Qemu-devel] [PATCH RESEND 43/50] blockdev: Implement change with basic operations, Max Reitz, 2015/01/27
- [Qemu-devel] [PATCH RESEND 47/50] blockdev: Add read-only option to blockdev-change-medium, Max Reitz, 2015/01/27
- [Qemu-devel] [PATCH RESEND 44/50] block: Inquire tray state before tray-moved events, Max Reitz, 2015/01/27
- [Qemu-devel] [PATCH RESEND 49/50] iotests: More options for VM.add_drive(), Max Reitz, 2015/01/27
- Re: [Qemu-devel] [PATCH RESEND 49/50] iotests: More options for VM.add_drive(),
Eric Blake <=
- [Qemu-devel] [PATCH RESEND 50/50] iotests: Add test for change-related QMP commands, Max Reitz, 2015/01/27
- [Qemu-devel] [PATCH RESEND 48/50] hmp: Add read-only option to change command, Max Reitz, 2015/01/27
- [Qemu-devel] [PATCH RESEND 03/50] hw/block/fdc: Implement tray status, Max Reitz, 2015/01/27