conf: No routable interface for IPv4 or IPv6 is informational, not a warning
...Podman users might get confused by the fact that if we can't find a default route for a given IP version, we'll report that as a warning message and possibly just before actual error messages. However, a lack of routable interface for IPv4 or IPv6 can be a normal circumstance: don't warn about it, just state that as informational message, if those are displayed (they're not in non-error paths in Podman, for example). While at it, make it clear that we're disabling IPv4 or IPv6 if there's no routable interface for the corresponding IP version. Reported-by: Paul Holzinger <pholzing@redhat.com> Link: https://github.com/containers/podman/pull/21563#issuecomment-1937024642 Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
This commit is contained in:
parent
8f3f8e190c
commit
338b6321ac
1 changed files with 2 additions and 2 deletions
4
conf.c
4
conf.c
|
@ -565,7 +565,7 @@ static unsigned int conf_ip4(unsigned int ifi,
|
||||||
ifi = nl_get_ext_if(nl_sock, AF_INET);
|
ifi = nl_get_ext_if(nl_sock, AF_INET);
|
||||||
|
|
||||||
if (!ifi) {
|
if (!ifi) {
|
||||||
warn("No external routable interface for IPv4");
|
info("No routable interface for IPv4: IPv4 is disabled");
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -637,7 +637,7 @@ static unsigned int conf_ip6(unsigned int ifi,
|
||||||
ifi = nl_get_ext_if(nl_sock, AF_INET6);
|
ifi = nl_get_ext_if(nl_sock, AF_INET6);
|
||||||
|
|
||||||
if (!ifi) {
|
if (!ifi) {
|
||||||
warn("No external routable interface for IPv6");
|
info("No routable interface for IPv6: IPv6 is disabled");
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue