[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 5/5] tests: add functional test validating ip
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH v4 5/5] tests: add functional test validating ipv4/ipv6 address flag handling |
Date: |
Mon, 26 Jun 2017 21:10:19 -0500 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.0 |
On 06/16/2017 05:12 AM, Daniel P. Berrange wrote:
> The semantics around handling ipv4=on|off & ipv6=on|off are quite
> subtle to understand in combination with the various hostname addresses
> and backend types. Introduce a massive test matrix that launches QEMU
> and validates the ability to connect a client on each protocol as
> appropriate.
>
> The test requires that the host has ability to bind to both :: and
> 0.0.0.0, on port 9000. If either protocol is not available, or if
> something is already listening on that port the test will skip.
>
> Although it isn't using the QTest APIs, it expects the
> QTEST_QEMU_BINARY env variable to be set.
I note that on failure, v3 created test-sockets-proto.pid in the current
working directory (the top level, if I ran 'make check-qtest') rather
than under the tests/ subdirectory or even better under a scratch
location that gets automatically cleaned up regardless of failure mode.
Since v3 failed for me, but v4 passes (and cleans up on success), I
can't say if that is still a problem in v4.
Otherwise, the diff between v3 and v4 looks sane, and it passes my
tests, so you can add:
Reviewed-by: Eric Blake <address@hidden>
Tested-by: Eric Blake <address@hidden>
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH v4 0/5] Fix handling of IPv4/IPv6 dual stack, Daniel P. Berrange, 2017/06/16
- [Qemu-devel] [PATCH v4 1/5] sockets: ensure we can bind to both ipv4 & ipv6 separately, Daniel P. Berrange, 2017/06/16
- [Qemu-devel] [PATCH v4 2/5] sockets: don't block IPv4 clients when listening on "::", Daniel P. Berrange, 2017/06/16
- [Qemu-devel] [PATCH v4 4/5] io: preserve ipv4/ipv6 flags when resolving InetSocketAddress, Daniel P. Berrange, 2017/06/16
- [Qemu-devel] [PATCH v4 3/5] sockets: ensure we don't accept IPv4 clients when IPv4 is disabled, Daniel P. Berrange, 2017/06/16
- [Qemu-devel] [PATCH v4 5/5] tests: add functional test validating ipv4/ipv6 address flag handling, Daniel P. Berrange, 2017/06/16
- Re: [Qemu-devel] [PATCH v4 5/5] tests: add functional test validating ipv4/ipv6 address flag handling,
Eric Blake <=