Commits on Oct 25, 2024

1. bgpd: fix prefix same as nexthop in label per nexthop …

   When a prefix is imported using the "network" command under a vrf, which
   is a connected prefix, and in the context of label allocation per
   nexthop:
   
   ..
   >router bgp 1 vrf vrf1
   > address-family ipv4 unicast
   >  redistribute static
   >  network 172.16.0.1/32  <--- connected network
   >  network 192.168.106.0/29
   >  label vpn export auto
   >  label vpn export allocation-mode per-nexthop
   ..
   
   We encounter an MPLS entry where the nexthop is the prefix itself:
   
   > 18             BGP   172.16.0.1     -
   
   Actually, when using the "network" command, a bnc context is used, but
   it is filled by using the prefix itself instead of the nexthop for other
   BGP updates. Consequently, when picking up the original nexthop for
   label allocation, the function behaves incorrectly. Instead ensure that
   the nexthop type of bnc->nexthop is not a nexthop_ifindex; otherwise
   fallback to the per vrf label.
   
   Update topotests.
   
   Signed-off-by: Loïc Sang <loic.sang@6wind.com>

   Loïc Sang committed Oct 25, 2024
   Configuration menu

   • View commit details
   • Copy full SHA for 6975228
   • Browse repository at this point

   Copy the full SHA

   6975228 View commit details

   Browse the repository at this point in the history

2. topotests: supress bgp_vpnv4/6_per_nexthop_label warning …

   WARNING: topo: Waiting time is too small
   (count=10, wait=0.5), using default values (count=20, wait=3)
   
   Supress warning by inscreasing wait time.
   
   Signed-off-by: Loïc Sang <loic.sang@6wind.com>

   Loïc Sang committed Oct 25, 2024
   Configuration menu

   • View commit details
   • Copy full SHA for 3c2949e
   • Browse repository at this point

   Copy the full SHA

   3c2949e View commit details

   Browse the repository at this point in the history