Skip to content

Commit

Permalink
tests: Check if OAD peers can exchange non-transitive extended commun…
Browse files Browse the repository at this point in the history
…ities

Signed-off-by: Donatas Abraitis <[email protected]>
  • Loading branch information
ton31337 committed Jan 21, 2025
1 parent f2759c4 commit de90b70
Show file tree
Hide file tree
Showing 2 changed files with 39 additions and 0 deletions.
6 changes: 6 additions & 0 deletions tests/topotests/bgp_oad/r3/frr.conf
Original file line number Diff line number Diff line change
Expand Up @@ -7,16 +7,22 @@ int r3-eth0
!
router bgp 65003
no bgp ebgp-requires-policy
no bgp network import-check
neighbor 192.168.2.2 remote-as external
neighbor 192.168.2.2 timers 1 3
neighbor 192.168.2.2 timers connect 1
neighbor 192.168.2.2 oad
!
address-family ipv4 unicast
network 10.10.10.20/32 route-map static
redistribute connected route-map connected
exit-address-family
!
route-map connected permit 10
set local-preference 123
set metric 123
!
route-map static permit 10
set extcommunity bandwidth 100 non-transitive
exit
!
33 changes: 33 additions & 0 deletions tests/topotests/bgp_oad/test_bgp_oad.py
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,7 @@ def test_bgp_oad():
r2 = tgen.gears["r2"]
r3 = tgen.gears["r3"]
r4 = tgen.gears["r4"]
r5 = tgen.gears["r5"]

def _bgp_converge():
output = json.loads(r1.vtysh_cmd("show bgp ipv4 unicast 10.10.10.10/32 json"))
Expand Down Expand Up @@ -121,6 +122,38 @@ def _bgp_check_no_export(router, arg=[{"valid": True}]):
_, result = topotest.run_and_expect(test_func, None, count=30, wait=1)
assert result is None, "10.10.10.1/32 should not be advertised to r4 (not OAD peer)"

def _bgp_check_non_transitive_extended_community(
router, arg={"string": "LB:65003:12500000 (100.000 Mbps)"}
):
output = json.loads(
router.vtysh_cmd("show bgp ipv4 unicast 10.10.10.20/32 json")
)
expected = {
"paths": [
{
"extendedCommunity": arg,
}
]
}
return topotest.json_cmp(output, expected)

test_func = functools.partial(
_bgp_check_non_transitive_extended_community,
r4,
)
_, result = topotest.run_and_expect(test_func, None, count=30, wait=1)
assert (
result is None
), "10.10.10.20/32 should be received at r4 with non-transitive extended community"

test_func = functools.partial(
_bgp_check_non_transitive_extended_community, r5, None
)
_, result = topotest.run_and_expect(test_func, None, count=30, wait=1)
assert (
result is None
), "10.10.10.20/32 should NOT be received at r5 with non-transitive extended community"


if __name__ == "__main__":
args = ["-s"] + sys.argv[1:]
Expand Down

0 comments on commit de90b70

Please sign in to comment.