3:32 p.m.
I started looking into this after seeing
FAIL: virnetsockettest
======================
TEST: virnetsockettest
1) Socket TCP/IPv4 Accept ... libvirt: XML-RPC error : Unable to
resolve address '127.0.0.1' service '5672': Address family for hostname
not supported
FAILED
2) Socket TCP/IPv6 Accept ... OK
3) Socket TCP/IPv4+IPv6 Accept ... OK
4) Socket TCP/IPv4+IPv6 Accept ... OK
5) Socket UNIX Accept ... OK
6) Socket UNIX Addrs ... OK
7) Socket External Command /dev/zero ... OK
8) Socket External Command /dev/does-not-exist ... libvirt: XML-RPC error : End of
file while reading data: /bin/cat: /dev/does-not-exist: No such file or directory:
Input/output error
OK
9) SSH test 1 ... OK
10) SSH test 2 ... OK
11) SSH test 3 ... OK
12) SSH test 4 ... libvirt: XML-RPC error : End of
file while reading data: Cannot connect to host nosuchhost: Input/output error
OK
13) SSH test 5 ... libvirt: XML-RPC error : End of
file while reading data: : Input/output error
OK
14) SSH test 6 ... OK
15) SSH test 7 ... OK
during a Debian package build.
Full log:
https://buildd.debian.org/status/fetch.php?pkg=libvirt&arch=armel&...
Just a few days ago, this issue was discussed in
https://lists.debian.org/debian-devel/2020/07/msg00070.html
and libvirt was mentioned explicitly as a package that could be
affected by it.
Reproducing the issue is quite simple: just remove all IPv4
addresses *except* the one assigned to 'lo', and you'll see it
immediately. The problem seems to be that, in this configuration,
virNetSocketCheckProtocols() reports that both IPv4 and IPv6 are
available, but XML-RPC is apparently not happy with that.
I'm pretty bad at networking, so I'm reporting this in the hope that
someone else will figure it out O:-)
Anyway, there's another scenario that I found while digging: if you
remove *all* IPv4 addresses, *including* the one assigned to 'lo',
you get a different failure. My patch addresses this last one.
Andrea Bolognani (2):
tests: Don't assume IPv4 connectivity is available
tests: Minimize variable scope
tests/virnetsockettest.c | 23 ++++++++++++++---------
1 file changed, 14 insertions(+), 9 deletions(-)
--
2.25.4
3:32 p.m.
New subject: [libvirt PATCH 1/2] tests: Don't assume IPv4 connectivity is available
If the host doesn't have a single IPv4 address assigned to any of
its interfaces, not even the loopback one, then virnetsockettest
will fail with
Cannot identify IPv4/6 availability
because, while the IPv6 bind attempt is conditional, the IPv4 one
is not, and in this case it will always fail.
This commit is better viewed with 'git show -w'.
Signed-off-by: Andrea Bolognani <abologna(a)redhat.com>
---
tests/virnetsockettest.c | 20 ++++++++++++--------
1 file changed, 12 insertions(+), 8 deletions(-)
diff --git a/tests/virnetsockettest.c b/tests/virnetsockettest.c
index 78fb9cbffd..f56e623cb3 100644
--- a/tests/virnetsockettest.c
+++ b/tests/virnetsockettest.c
@@ -56,8 +56,10 @@ checkProtocols(bool *hasIPv4, bool *hasIPv6,
for (i = 0; i < 50; i++) {
int only = 1;
- if ((s4 = socket(AF_INET, SOCK_STREAM, 0)) < 0)
- goto cleanup;
+ if (*hasIPv4) {
+ if ((s4 = socket(AF_INET, SOCK_STREAM, 0)) < 0)
+ goto cleanup;
+ }
if (*hasIPv6) {
if ((s6 = socket(AF_INET6, SOCK_STREAM, 0)) < 0)
@@ -77,13 +79,15 @@ checkProtocols(bool *hasIPv4, bool *hasIPv6,
in6.sin6_port = htons(BASE_PORT + i);
in6.sin6_addr = in6addr_loopback;
- if (bind(s4, (struct sockaddr *)&in4, sizeof(in4)) < 0) {
- if (errno == EADDRINUSE) {
- VIR_FORCE_CLOSE(s4);
- VIR_FORCE_CLOSE(s6);
- continue;
+ if (*hasIPv4) {
+ if (bind(s4, (struct sockaddr *)&in4, sizeof(in4)) < 0) {
+ if (errno == EADDRINUSE) {
+ VIR_FORCE_CLOSE(s4);
+ VIR_FORCE_CLOSE(s6);
+ continue;
+ }
+ goto cleanup;
}
- goto cleanup;
}
if (*hasIPv6) {
--
2.25.4
3:33 p.m.
New subject: [libvirt PATCH 2/2] tests: Minimize variable scope
Signed-off-by: Andrea Bolognani <abologna(a)redhat.com>
---
tests/virnetsockettest.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/tests/virnetsockettest.c b/tests/virnetsockettest.c
index f56e623cb3..96c582216c 100644
--- a/tests/virnetsockettest.c
+++ b/tests/virnetsockettest.c
@@ -55,13 +55,14 @@ checkProtocols(bool *hasIPv4, bool *hasIPv6,
return -1;
for (i = 0; i < 50; i++) {
- int only = 1;
if (*hasIPv4) {
if ((s4 = socket(AF_INET, SOCK_STREAM, 0)) < 0)
goto cleanup;
}
if (*hasIPv6) {
+ int only = 1;
+
if ((s6 = socket(AF_INET6, SOCK_STREAM, 0)) < 0)
goto cleanup;
--
2.25.4
5:38 a.m.
On 7/14/20 10:32 PM, Andrea Bolognani wrote:
I started looking into this after seeing
FAIL: virnetsockettest
======================
TEST: virnetsockettest
1) Socket TCP/IPv4 Accept ... libvirt: XML-RPC error : Unable
to resolve address '127.0.0.1' service '5672': Address family for hostname
not supported
FAILED
during a Debian package build.
Full log:
https://buildd.debian.org/status/fetch.php?pkg=libvirt&arch=armel&...
Just a few days ago, this issue was discussed in
https://lists.debian.org/debian-devel/2020/07/msg00070.html
and libvirt was mentioned explicitly as a package that could be
affected by it.
Indeed. I'm able to reproduce and working on a fix as we speak.
The problem is that our test assumes that if there is an interface with
IPv4 address (as returned by getifaddrs()) then getaddrinfo() of an IPv4
address succeeds. We've seen with IPv6 that it is not true - that's why
virNetSocketCheckProtocols() does explicit getaddrinfo() for an IPv6
address. We need to do the same for IPv4.
Anyway, for these two:
Reviewed-by: Michal Privoznik <mprivozn(a)redhat.com>
Michal
6:08 a.m.
On Wed, 2020-07-15 at 12:38 +0200, Michal Privoznik wrote:
On 7/14/20 10:32 PM, Andrea Bolognani wrote:
> I started looking into this after seeing
>
> FAIL: virnetsockettest
> ======================
>
> TEST: virnetsockettest
> 1) Socket TCP/IPv4 Accept ... libvirt: XML-RPC error :
Unable to resolve address '127.0.0.1' service '5672': Address family for
hostname not supported
> FAILED
> during a Debian package build.
>
> Full log:
>
>
https://buildd.debian.org/status/fetch.php?pkg=libvirt&arch=armel&...
>
> Just a few days ago, this issue was discussed in
>
> https://lists.debian.org/debian-devel/2020/07/msg00070.html
>
> and libvirt was mentioned explicitly as a package that could be
> affected by it.
Indeed. I'm able to reproduce and working on a fix as we speak.
The problem is that our test assumes that if there is an interface with
IPv4 address (as returned by getifaddrs()) then getaddrinfo() of an IPv4
address succeeds. We've seen with IPv6 that it is not true - that's why
virNetSocketCheckProtocols() does explicit getaddrinfo() for an IPv6
address. We need to do the same for IPv4.
That's what I thought needed to happen, but I lack the networking
know-how to write the patches myself O:-)
Anyway, for these two:
Reviewed-by: Michal Privoznik <mprivozn(a)redhat.com>
Thanks, pushed now.
--
Andrea Bolognani / Red Hat / Virtualization
1591
days inactive
1592
days old
4 comments
2 participants
participants (2)
-
Andrea Bolognani -
Michal Privoznik