diff options
| author | Philippe Guibert <philippe.guibert@6wind.com> | 2024-05-14 11:51:50 +0200 |
|---|---|---|
| committer | Philippe Guibert <philippe.guibert@6wind.com> | 2024-05-14 13:51:24 +0200 |
| commit | e265b16f83e224f349b31f15227b3a99c30b8ecb (patch) | |
| tree | 08f12a3c60ce460dd22eb3da8a7a33393a25f7b5 | |
| parent | 281c891f167c198d0cdd90f54e06f2f30e403e87 (diff) | |
bgpd: fix colored routes not installed after a switchover
On a multihomed setup with colored bgp updates, when the primary
PE goes offline, only a small subset of colored bgp routes are
not switching to the secondary pe.
When a switchover happens, due to a remote IP becoming unreachable,
some nexthop tracking down notifications are sent, but those messages
are completely ignored for colored bgp updates.
The original code has been thought for mounting up the SR-TE service,
when IP reachability is ok, but not when services goes offline.
Fix this by extending the down notification mechanism for colored routes
too.
Fixes: 545aeef1d13e ("bgpd: extend the NHT code to understand SR-TE colors")
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
| -rw-r--r-- | bgpd/bgp_nht.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/bgpd/bgp_nht.c b/bgpd/bgp_nht.c index fca3da9386..79f62ef609 100644 --- a/bgpd/bgp_nht.c +++ b/bgpd/bgp_nht.c @@ -978,8 +978,7 @@ void bgp_nexthop_update(struct vrf *vrf, struct prefix *match, frr_each (bgp_nexthop_cache, &bgp->nexthop_cache_table[afi], bnc_iter) { if (!prefix_same(match, &bnc_iter->prefix) || - bnc_iter->srte_color == 0 || - CHECK_FLAG(bnc_iter->flags, BGP_NEXTHOP_VALID)) + bnc_iter->srte_color == 0) continue; bgp_process_nexthop_update(bnc_iter, nhr, false); |