Merge pull request #16305 from opensourcerouting/fix/bgp_confederatio…

…n_rfc8212 bgpd: Ignore RFC8212 for BGP Confederations
FRRouting · Jul 1, 2024 · 0ed2b5a · 0ed2b5a
2 parents 0727e97 + dd6a679
commit 0ed2b5a
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/bgpd/bgp_route.c b/bgpd/bgp_route.c
@@ -6333,7 +6333,8 @@ void bgp_set_stale_route(struct peer *peer, afi_t afi, safi_t safi)
 
 bool bgp_outbound_policy_exists(struct peer *peer, struct bgp_filter *filter)
 {
-	if (peer->sort == BGP_PEER_IBGP || peer->sub_sort == BGP_PEER_EBGP_OAD)
+	if (peer->sort == BGP_PEER_CONFED || peer->sort == BGP_PEER_IBGP ||
+	    peer->sub_sort == BGP_PEER_EBGP_OAD)
 		return true;
 
 	if (peer->sort == BGP_PEER_EBGP &&
@@ -6346,7 +6347,8 @@ bool bgp_outbound_policy_exists(struct peer *peer, struct bgp_filter *filter)
 
 bool bgp_inbound_policy_exists(struct peer *peer, struct bgp_filter *filter)
 {
-	if (peer->sort == BGP_PEER_IBGP || peer->sub_sort == BGP_PEER_EBGP_OAD)
+	if (peer->sort == BGP_PEER_CONFED || peer->sort == BGP_PEER_IBGP ||
+	    peer->sub_sort == BGP_PEER_EBGP_OAD)
 		return true;
 
 	if (peer->sort == BGP_PEER_EBGP

diff --git a/tests/topotests/bgp_confed1/r2/bgpd.conf b/tests/topotests/bgp_confed1/r2/bgpd.conf
@@ -4,15 +4,16 @@
 !debug bgp updates out
 !
 router bgp 200
- no bgp ebgp-requires-policy
  bgp confederation identifier 300
  bgp confederation peers 300
  neighbor 192.0.2.1 remote-as 100
  neighbor 192.0.2.18 remote-as 300
  !
  address-family ipv4 unicast
   network 203.0.113.16/28
+  neighbor 192.0.2.1 route-map any in
+  neighbor 192.0.2.1 route-map any out
   neighbor 192.0.2.18 default-originate
  exit-address-family
 !
-
+route-map any permit 10