router bgp 65001
no bgp ebgp-requires-policy
no bgp network import-check
+ bgp route-reflector allow-outbound-policy
neighbor 192.168.24.2 remote-as internal
neighbor 192.168.24.2 timers 1 3
neighbor 192.168.24.2 timers connect 1
neighbor 10.0.0.6 timers connect 1
neighbor 10.0.0.6 update-source lo
address-family ipv4
- redistribute connected
- redistribute ospf
+ redistribute connected route-map connected-to-bgp
+ neighbor 192.168.24.2 route-map set-nexthop out
exit-address-family
!
+! Two OSPF domains should be isolated - otherwise the connected routes
+! on r4 would be advertised to r3 (via r4 -> r6 -> r5 -> r3), and can
+! mess up bgp bestpath calculation (igp metrics for the BGP nexthops).
+!
+route-map connected-to-bgp permit 10
+ set community no-advertise
+!
+route-map set-nexthop permit 10
+ set ip next-hop peer-address
+exit
+!
router bgp 65001
no bgp ebgp-requires-policy
no bgp network import-check
+ bgp route-reflector allow-outbound-policy
neighbor 192.168.35.3 remote-as internal
neighbor 192.168.35.3 timers 1 3
neighbor 192.168.35.3 timers connect 1
neighbor 10.0.0.6 timers connect 1
neighbor 10.0.0.6 update-source lo
address-family ipv4
- redistribute connected
- redistribute ospf
+ redistribute connected route-map connected-to-bgp
+ neighbor 192.168.35.3 route-map set-nexthop out
exit-address-family
!
+! Two OSPF domains should be isolated - otherwise the connected routes
+! on r5 would be advertised to r2 (via r5 -> r6 -> r4 -> r2), and can
+! mess up bgp bestpath calculation (igp metrics for the BGP nexthops).
+!
+route-map connected-to-bgp permit 10
+ set community no-advertise
+!
+route-map set-nexthop permit 10
+ set ip next-hop peer-address
+exit
+!
r6 receives those routes with aigp-metric TLV.
r2 and r3 receives those routes with aigp-metric TLV increased by 20,
-and 30 appropriately.
+and 10 appropriately.
-r1 receives routes with aigp-metric TLV 111,131 and 112,132 appropriately.
+r1 receives routes with aigp-metric TLV 81, 91 and 82, 92 respectively.
"""
import os
expected = {
"paths": [
{
- "aigpMetric": 111,
+ "aigpMetric": 81,
"valid": True,
- "nexthops": [{"hostname": "r3", "accessible": True}],
+ "nexthops": [
+ {
+ "ip": "10.0.0.3",
+ "hostname": "r3",
+ "metric": 30,
+ "accessible": True,
+ }
+ ],
},
{
- "aigpMetric": 131,
+ "aigpMetric": 91,
"valid": True,
- "bestpath": {"selectionReason": "Neighbor IP"},
- "nexthops": [{"hostname": "r2", "accessible": True}],
+ "bestpath": {"selectionReason": "IGP Metric"},
+ "nexthops": [
+ {
+ "ip": "10.0.0.2",
+ "hostname": "r2",
+ "metric": 10,
+ "accessible": True,
+ }
+ ],
},
]
}
"10.0.0.71/32": {
"paths": [
{
- "aigpMetric": 111,
- "bestpath": {"selectionReason": "AIGP"},
+ "aigpMetric": 81,
"valid": True,
- "nexthops": [{"hostname": "r3", "accessible": True}],
+ "nexthops": [
+ {
+ "ip": "10.0.0.3",
+ "hostname": "r3",
+ "metric": 30,
+ "accessible": True,
+ }
+ ],
},
{
- "aigpMetric": 131,
+ "aigpMetric": 91,
"valid": True,
- "nexthops": [{"hostname": "r2", "accessible": True}],
+ "bestpath": {"selectionReason": "AIGP"},
+ "nexthops": [
+ {
+ "ip": "10.0.0.2",
+ "hostname": "r2",
+ "metric": 10,
+ "accessible": True,
+ }
+ ],
},
],
},
"10.0.0.72/32": {
"paths": [
{
- "aigpMetric": 112,
- "bestpath": {"selectionReason": "AIGP"},
+ "aigpMetric": 82,
"valid": True,
- "nexthops": [{"hostname": "r3", "accessible": True}],
+ "nexthops": [
+ {
+ "ip": "10.0.0.3",
+ "hostname": "r3",
+ "metric": 30,
+ "accessible": True,
+ }
+ ],
},
{
- "aigpMetric": 132,
+ "aigpMetric": 92,
"valid": True,
- "nexthops": [{"hostname": "r2", "accessible": True}],
+ "bestpath": {"selectionReason": "AIGP"},
+ "nexthops": [
+ {
+ "ip": "10.0.0.2",
+ "hostname": "r2",
+ "metric": 10,
+ "accessible": True,
+ }
+ ],
},
],
},
_, result = topotest.run_and_expect(test_func, None, count=60, wait=1)
assert result is None, "aigp-metric for 10.0.0.72/32 is not 72"
- # r2, 10.0.0.71/32 with aigp-metric 101 (71 + 30)
- test_func = functools.partial(_bgp_check_aigp_metric, r2, "10.0.0.71/32", 101)
+ # r2, 10.0.0.71/32 with aigp-metric 101 (71 + 20)
+ test_func = functools.partial(_bgp_check_aigp_metric, r2, "10.0.0.71/32", 91)
_, result = topotest.run_and_expect(test_func, None, count=60, wait=1)
- assert result is None, "aigp-metric for 10.0.0.71/32 is not 101"
+ assert result is None, "aigp-metric for 10.0.0.71/32 is not 91"
- # r3, 10.0.0.72/32 with aigp-metric 92 (72 + 20)
- test_func = functools.partial(_bgp_check_aigp_metric, r3, "10.0.0.72/32", 92)
+ # r3, 10.0.0.72/32 with aigp-metric 92 (72 + 10)
+ test_func = functools.partial(_bgp_check_aigp_metric, r3, "10.0.0.72/32", 82)
_, result = topotest.run_and_expect(test_func, None, count=60, wait=1)
- assert result is None, "aigp-metric for 10.0.0.72/32 is not 92"
+ assert result is None, "aigp-metric for 10.0.0.72/32 is not 82"
- # r1, check if AIGP is considered in best-path selection (lowest wins)
+ # r1, check if AIGP is considered in best-path selection (lowest wins: aigp + nexthop-metric)
test_func = functools.partial(_bgp_check_aigp_metric_bestpath)
_, result = topotest.run_and_expect(test_func, None, count=60, wait=1)
assert result is None, "AIGP attribute is not considered in best-path selection"
projects / matthieu / frr.git / commitdiff