Merge pull request FRRouting#16313 from FRRouting/mergify/bp/stable/1…

…0.0/pr-16305 bgpd: Ignore RFC8212 for BGP Confederations (backport FRRouting#16305)
opensourcerouting · Jul 2, 2024 · f49ef22 · f49ef22
2 parents edc1208 + 926554d
commit f49ef22
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
@@ -6024,7 +6024,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 &&
@@ -6037,7 +6038,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