frr icon indicating copy to clipboard operation
frr copied to clipboard

bgpd: Fix BGP to update correct v6 next-hop globally

Open routingrocks opened this issue 1 year ago • 6 comments

Issue: In a dual-stack IPv4/IPv6 BGP session, after the interface is deactivated (ifdown) and then reloaded (ifreload), the global next-hop within BGP is incorrectly updated. Currently, only unnumbered neighbors are handled. We also need to handle an IPv4 neighbor when IPv6 AFI-SAFI is enabled.

Fix: When the primary global v6 unicast address is received from Zebra, BGP updates the peer's global next-hop with the correct address and send update to its negibors.

Testing: UT, and TestEbgpMultihop tests

UT logs: with fix: root@:mgmt:/tmp# ifdown swp1s0; ifreload -a root@:mgmt:/tmp# vtysh -c "show ip bgp vrf all neighbors" | grep Nexthop Nexthop: 20.1.2.101 Nexthop global: 2001:10:1:2::101 Nexthop local: fe80::7efe:90ff:fefa:e158

without fix after ifreload: Nexthop: 20.1.2.101 Nexthop global: :: Nexthop local: fe80::7efe:90ff:fefa:e158

Ticket: # Signed-off-by: Rajesh Varatharaj [email protected] and Pooja Doijode [email protected]

routingrocks avatar May 03 '24 19:05 routingrocks

@Mergifyio backport stable/10.0

ton31337 avatar May 04 '24 09:05 ton31337

backport stable/10.0

🟠 Waiting for conditions to match

  • [ ] merged [📌 backport requirement]

mergify[bot] avatar May 04 '24 09:05 mergify[bot]

@routingrocks could you rebase?

ton31337 avatar May 10 '24 19:05 ton31337

This pull request has conflicts, please resolve those before we can evaluate the pull request.

github-actions[bot] avatar May 10 '24 19:05 github-actions[bot]

Normally, the issue is fixed by https://github.com/FRRouting/frr/pull/15614, which is now merged Could you test your scenario with the latest master branch?

louis-6wind avatar May 13 '24 09:05 louis-6wind

@frrbot autoclose in 1 month

riw777 avatar May 28 '24 14:05 riw777