opensourcerouting / frr

Free Range Routing Protocol Suite
Other
37 stars 14 forks source link

bgpd: do not unregister for prefix nexthop updates if nh exists #29

Closed pjdruddy closed 4 years ago

pjdruddy commented 4 years ago

since the addition of srte_color to the comparison for bgp nexthops it is possible to have several nexthops per prefix but since zebra only stores a per prefix registration we should not unregister for nh notifications for a prefix until all the nexthops for that prefix have been deleted. Otherwise we can get into a deadlock situation where BGP thinks we have registered but we have unregistered from zebra.

Signed-off-by: Pat Ruddy pat@voltanet.io

GalaxyGorilla commented 4 years ago

CI:rerun