laforge has submitted this change. (
https://gerrit.osmocom.org/c/osmo-uecups/+/27021 )
Change subject: daemon: log more details when netdev_add_defaultroute() fails
......................................................................
daemon: log more details when netdev_add_defaultroute() fails
Change-Id: I4ac09ec528e6e9169cca813995ba0db583b212a0
Related: SYS#5602
---
M daemon/tun_device.c
1 file changed, 4 insertions(+), 2 deletions(-)
Approvals:
Jenkins Builder: Verified
laforge: Looks good to me, approved
diff --git a/daemon/tun_device.c b/daemon/tun_device.c
index b1bd7fc..7d1948f 100644
--- a/daemon/tun_device.c
+++ b/daemon/tun_device.c
@@ -290,13 +290,15 @@
if (tun->netns_name) {
rc = netdev_add_defaultroute(tun->nl, tun->ifindex, AF_INET);
if (rc < 0)
- LOGTUN(tun, LOGL_ERROR, "Cannot add IPv4 default route\n");
+ LOGTUN(tun, LOGL_ERROR, "Cannot add IPv4 default route "
+ "(rc=%d): %s\n", rc, nl_geterror(rc));
else
LOGTUN(tun, LOGL_INFO, "Added IPv4 default route\n");
rc = netdev_add_defaultroute(tun->nl, tun->ifindex, AF_INET6);
if (rc < 0)
- LOGTUN(tun, LOGL_ERROR, "Cannot add IPv6 default route\n");
+ LOGTUN(tun, LOGL_ERROR, "Cannot add IPv6 default route "
+ "(rc=%d): %s\n", rc, nl_geterror(rc));
else
LOGTUN(tun, LOGL_INFO, "Added IPv6 default route\n");
}
--
To view, visit
https://gerrit.osmocom.org/c/osmo-uecups/+/27021
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-uecups
Gerrit-Branch: master
Gerrit-Change-Id: I4ac09ec528e6e9169cca813995ba0db583b212a0
Gerrit-Change-Number: 27021
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-MessageType: merged