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 /bgpd | |
| parent | Merge pull request #16003 from pguibert6WIND/fix_colored_nexthop (diff) | |
| download | frr-e265b16f83e224f349b31f15227b3a99c30b8ecb.tar.xz frr-e265b16f83e224f349b31f15227b3a99c30b8ecb.zip | |
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>
Diffstat (limited to 'bgpd')
| -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 fca3da938..79f62ef60 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); |