bgpd: Add some color to why nexthop_set failed

We are seeing some frequent test failures with setting the nexthop correctly. At this point in time, I have no idea what is going wrong, but I don't have a bunch of information either, so let's add the local and remote values. Signed-off-by: Donald Sharp <[email protected]>
FRRouting · Jun 12, 2023 · c1ce67e · c1ce67e
1 parent 6ec1065
commit c1ce67e
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/bgpd/bgp_network.c b/bgpd/bgp_network.c
@@ -810,8 +810,9 @@ int bgp_getsockname(struct peer *peer)
 				   &peer->nexthop, peer)) {
 		flog_err(
 			EC_BGP_NH_UPD,
-			"%s: nexthop_set failed, resetting connection - intf %s",
-			peer->host,
+			"%s: nexthop_set failed, local: %pSUp remote: %pSUp update_if: %s resetting connection - intf %s",
+			peer->host, peer->su_local, peer->su_remote,
+			peer->update_if ? peer->update_if : "(None)",
 			peer->nexthop.ifp ? peer->nexthop.ifp->name
 					  : "(Unknown)");
 		return -1;