diff options
Diffstat (limited to 'tests/topotests/bgp_aigp')
22 files changed, 493 insertions, 0 deletions
diff --git a/tests/topotests/bgp_aigp/__init__.py b/tests/topotests/bgp_aigp/__init__.py new file mode 100644 index 0000000..e69de29 --- /dev/null +++ b/tests/topotests/bgp_aigp/__init__.py diff --git a/tests/topotests/bgp_aigp/r1/bgpd.conf b/tests/topotests/bgp_aigp/r1/bgpd.conf new file mode 100644 index 0000000..74a0215 --- /dev/null +++ b/tests/topotests/bgp_aigp/r1/bgpd.conf @@ -0,0 +1,12 @@ +router bgp 65001 + no bgp ebgp-requires-policy + no bgp network import-check + neighbor 10.0.0.2 remote-as internal + neighbor 10.0.0.2 update-source lo + neighbor 10.0.0.2 timers 1 3 + neighbor 10.0.0.2 timers connect 1 + neighbor 10.0.0.3 remote-as internal + neighbor 10.0.0.3 timers 1 3 + neighbor 10.0.0.3 timers connect 1 + neighbor 10.0.0.3 update-source lo +! diff --git a/tests/topotests/bgp_aigp/r1/ospfd.conf b/tests/topotests/bgp_aigp/r1/ospfd.conf new file mode 100644 index 0000000..38aa11d --- /dev/null +++ b/tests/topotests/bgp_aigp/r1/ospfd.conf @@ -0,0 +1,17 @@ +! +interface lo + ip ospf cost 10 +! +interface r1-eth0 + ip ospf dead-interval 4 + ip ospf hello-interval 1 + ip ospf cost 10 +! +interface r1-eth1 + ip ospf dead-interval 4 + ip ospf hello-interval 1 + ip ospf cost 30 +! +router ospf + router-id 10.0.0.1 + network 0.0.0.0/0 area 0 diff --git a/tests/topotests/bgp_aigp/r1/zebra.conf b/tests/topotests/bgp_aigp/r1/zebra.conf new file mode 100644 index 0000000..0ed22d3 --- /dev/null +++ b/tests/topotests/bgp_aigp/r1/zebra.conf @@ -0,0 +1,10 @@ +! +interface lo + ip address 10.0.0.1/32 +! +interface r1-eth0 + ip address 192.168.12.1/24 +! +interface r1-eth1 + ip address 192.168.13.1/24 +! diff --git a/tests/topotests/bgp_aigp/r2/bgpd.conf b/tests/topotests/bgp_aigp/r2/bgpd.conf new file mode 100644 index 0000000..4db4687 --- /dev/null +++ b/tests/topotests/bgp_aigp/r2/bgpd.conf @@ -0,0 +1,15 @@ +router bgp 65001 + no bgp ebgp-requires-policy + no bgp network import-check + neighbor 10.0.0.1 remote-as internal + neighbor 10.0.0.1 timers 1 3 + neighbor 10.0.0.1 timers connect 1 + neighbor 10.0.0.1 route-reflector-client + neighbor 192.168.24.4 remote-as internal + neighbor 192.168.24.4 timers 1 3 + neighbor 192.168.24.4 timers connect 1 + neighbor 192.168.24.4 aigp + address-family ipv4 + redistribute connected + exit-address-family +! diff --git a/tests/topotests/bgp_aigp/r2/ospfd.conf b/tests/topotests/bgp_aigp/r2/ospfd.conf new file mode 100644 index 0000000..ed31941 --- /dev/null +++ b/tests/topotests/bgp_aigp/r2/ospfd.conf @@ -0,0 +1,13 @@ +! +interface lo + ip ospf cost 10 +! +interface r2-eth0 + ip ospf dead-interval 4 + ip ospf hello-interval 1 + ip ospf cost 10 +! +router ospf + router-id 10.0.0.2 + network 192.168.12.0/24 area 0 + network 10.0.0.2/32 area 0 diff --git a/tests/topotests/bgp_aigp/r2/zebra.conf b/tests/topotests/bgp_aigp/r2/zebra.conf new file mode 100644 index 0000000..6d6a557 --- /dev/null +++ b/tests/topotests/bgp_aigp/r2/zebra.conf @@ -0,0 +1,10 @@ +! +interface lo + ip address 10.0.0.2/32 +! +interface r2-eth0 + ip address 192.168.12.2/24 +! +interface r2-eth1 + ip address 192.168.24.2/24 +! diff --git a/tests/topotests/bgp_aigp/r3/bgpd.conf b/tests/topotests/bgp_aigp/r3/bgpd.conf new file mode 100644 index 0000000..5ab712e --- /dev/null +++ b/tests/topotests/bgp_aigp/r3/bgpd.conf @@ -0,0 +1,15 @@ +router bgp 65001 + no bgp ebgp-requires-policy + no bgp network import-check + neighbor 10.0.0.1 remote-as internal + neighbor 10.0.0.1 timers 1 3 + neighbor 10.0.0.1 timers connect 1 + neighbor 10.0.0.1 route-reflector-client + neighbor 192.168.35.5 remote-as internal + neighbor 192.168.35.5 timers 1 3 + neighbor 192.168.35.5 timers connect 1 + neighbor 192.168.35.5 aigp + address-family ipv4 + redistribute connected + exit-address-family +! diff --git a/tests/topotests/bgp_aigp/r3/ospfd.conf b/tests/topotests/bgp_aigp/r3/ospfd.conf new file mode 100644 index 0000000..f971ad6 --- /dev/null +++ b/tests/topotests/bgp_aigp/r3/ospfd.conf @@ -0,0 +1,13 @@ +! +interface lo + ip ospf cost 10 +! +interface r3-eth0 + ip ospf dead-interval 4 + ip ospf hello-interval 1 + ip ospf cost 30 +! +router ospf + router-id 10.0.0.3 + network 192.168.13.0/24 area 0 + network 10.0.0.3/32 area 0 diff --git a/tests/topotests/bgp_aigp/r3/zebra.conf b/tests/topotests/bgp_aigp/r3/zebra.conf new file mode 100644 index 0000000..82c87d5 --- /dev/null +++ b/tests/topotests/bgp_aigp/r3/zebra.conf @@ -0,0 +1,10 @@ +! +interface lo + ip address 10.0.0.3/32 +! +interface r3-eth0 + ip address 192.168.13.3/24 +! +interface r3-eth1 + ip address 192.168.35.3/24 +! diff --git a/tests/topotests/bgp_aigp/r4/bgpd.conf b/tests/topotests/bgp_aigp/r4/bgpd.conf new file mode 100644 index 0000000..aa88bac --- /dev/null +++ b/tests/topotests/bgp_aigp/r4/bgpd.conf @@ -0,0 +1,17 @@ +router bgp 65001 + no bgp ebgp-requires-policy + no bgp network import-check + 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 192.168.24.2 aigp + neighbor 192.168.24.2 route-reflector-client + neighbor 10.0.0.6 remote-as internal + neighbor 10.0.0.6 timers 1 3 + neighbor 10.0.0.6 timers connect 1 + neighbor 10.0.0.6 update-source lo + address-family ipv4 + redistribute connected + redistribute ospf + exit-address-family +! diff --git a/tests/topotests/bgp_aigp/r4/ospfd.conf b/tests/topotests/bgp_aigp/r4/ospfd.conf new file mode 100644 index 0000000..c9e6796 --- /dev/null +++ b/tests/topotests/bgp_aigp/r4/ospfd.conf @@ -0,0 +1,13 @@ +! +interface lo + ip ospf cost 10 +! +interface r4-eth1 + ip ospf dead-interval 4 + ip ospf hello-interval 1 + ip ospf cost 20 +! +router ospf + router-id 10.0.0.4 + network 192.168.46.0/24 area 0 + network 10.0.0.4/32 area 0 diff --git a/tests/topotests/bgp_aigp/r4/zebra.conf b/tests/topotests/bgp_aigp/r4/zebra.conf new file mode 100644 index 0000000..5f544e8 --- /dev/null +++ b/tests/topotests/bgp_aigp/r4/zebra.conf @@ -0,0 +1,10 @@ +! +interface lo + ip address 10.0.0.4/32 +! +interface r4-eth0 + ip address 192.168.24.4/24 +! +interface r4-eth1 + ip address 192.168.46.4/24 +! diff --git a/tests/topotests/bgp_aigp/r5/bgpd.conf b/tests/topotests/bgp_aigp/r5/bgpd.conf new file mode 100644 index 0000000..4fde262 --- /dev/null +++ b/tests/topotests/bgp_aigp/r5/bgpd.conf @@ -0,0 +1,17 @@ +router bgp 65001 + no bgp ebgp-requires-policy + no bgp network import-check + 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 192.168.35.3 aigp + neighbor 192.168.35.3 route-reflector-client + neighbor 10.0.0.6 remote-as internal + neighbor 10.0.0.6 timers 1 3 + neighbor 10.0.0.6 timers connect 1 + neighbor 10.0.0.6 update-source lo + address-family ipv4 + redistribute connected + redistribute ospf + exit-address-family +! diff --git a/tests/topotests/bgp_aigp/r5/ospfd.conf b/tests/topotests/bgp_aigp/r5/ospfd.conf new file mode 100644 index 0000000..b853c74 --- /dev/null +++ b/tests/topotests/bgp_aigp/r5/ospfd.conf @@ -0,0 +1,13 @@ +! +interface lo + ip ospf cost 10 +! +interface r5-eth1 + ip ospf dead-interval 4 + ip ospf hello-interval 1 + ip ospf cost 10 +! +router ospf + router-id 10.0.0.5 + network 192.168.56.0/24 area 0 + network 10.0.0.5/32 area 0 diff --git a/tests/topotests/bgp_aigp/r5/zebra.conf b/tests/topotests/bgp_aigp/r5/zebra.conf new file mode 100644 index 0000000..69b8bf2 --- /dev/null +++ b/tests/topotests/bgp_aigp/r5/zebra.conf @@ -0,0 +1,10 @@ +! +interface lo + ip address 10.0.0.5/32 +! +interface r5-eth0 + ip address 192.168.35.5/24 +! +interface r5-eth1 + ip address 192.168.56.5/24 +! diff --git a/tests/topotests/bgp_aigp/r6/bgpd.conf b/tests/topotests/bgp_aigp/r6/bgpd.conf new file mode 100644 index 0000000..2faae77 --- /dev/null +++ b/tests/topotests/bgp_aigp/r6/bgpd.conf @@ -0,0 +1,20 @@ +router bgp 65001 + no bgp ebgp-requires-policy + no bgp network import-check + neighbor 10.0.0.4 remote-as internal + neighbor 10.0.0.4 timers 1 3 + neighbor 10.0.0.4 timers connect 1 + neighbor 10.0.0.4 route-reflector-client + neighbor 10.0.0.4 update-source lo + neighbor 10.0.0.5 remote-as internal + neighbor 10.0.0.5 timers 1 3 + neighbor 10.0.0.5 timers connect 1 + neighbor 10.0.0.5 route-reflector-client + neighbor 10.0.0.5 update-source lo + neighbor 192.168.67.7 remote-as internal + neighbor 192.168.67.7 timers 1 3 + neighbor 192.168.67.7 timers connect 1 + address-family ipv4 + redistribute ospf + exit-address-family +! diff --git a/tests/topotests/bgp_aigp/r6/ospfd.conf b/tests/topotests/bgp_aigp/r6/ospfd.conf new file mode 100644 index 0000000..46b2933 --- /dev/null +++ b/tests/topotests/bgp_aigp/r6/ospfd.conf @@ -0,0 +1,17 @@ +! +interface lo + ip ospf cost 10 +! +interface r6-eth0 + ip ospf dead-interval 4 + ip ospf hello-interval 1 + ip ospf cost 20 +! +interface r6-eth1 + ip ospf dead-interval 4 + ip ospf hello-interval 1 + ip ospf cost 10 +! +router ospf + router-id 10.0.0.6 + network 0.0.0.0/0 area 0 diff --git a/tests/topotests/bgp_aigp/r6/zebra.conf b/tests/topotests/bgp_aigp/r6/zebra.conf new file mode 100644 index 0000000..f8ca5f8 --- /dev/null +++ b/tests/topotests/bgp_aigp/r6/zebra.conf @@ -0,0 +1,13 @@ +! +interface lo + ip address 10.0.0.6/32 +! +interface r6-eth0 + ip address 192.168.46.6/24 +! +interface r6-eth1 + ip address 192.168.56.6/24 +! +interface r6-eth2 + ip address 192.168.67.6/24 +! diff --git a/tests/topotests/bgp_aigp/r7/bgpd.conf b/tests/topotests/bgp_aigp/r7/bgpd.conf new file mode 100644 index 0000000..639dcfe --- /dev/null +++ b/tests/topotests/bgp_aigp/r7/bgpd.conf @@ -0,0 +1,22 @@ +router bgp 65001 + no bgp ebgp-requires-policy + no bgp network import-check + neighbor 192.168.67.6 remote-as internal + neighbor 192.168.67.6 timers 1 3 + neighbor 192.168.67.6 timers connect 1 + address-family ipv4 + redistribute connected route-map rmap metric 71 + exit-address-family +! +ip prefix-list p71 seq 5 permit 10.0.0.71/32 +ip prefix-list p72 seq 5 permit 10.0.0.72/32 +! +route-map rmap permit 10 + match ip address prefix-list p71 + set aigp igp-metric +! +route-map rmap permit 20 + match ip address prefix-list p72 + set aigp 72 +exit +! diff --git a/tests/topotests/bgp_aigp/r7/zebra.conf b/tests/topotests/bgp_aigp/r7/zebra.conf new file mode 100644 index 0000000..4c05df8 --- /dev/null +++ b/tests/topotests/bgp_aigp/r7/zebra.conf @@ -0,0 +1,9 @@ +! +interface lo + ip address 10.0.0.7/32 + ip address 10.0.0.71/32 + ip address 10.0.0.72/32 +! +interface r7-eth0 + ip address 192.168.67.7/24 +! diff --git a/tests/topotests/bgp_aigp/test_bgp_aigp.py b/tests/topotests/bgp_aigp/test_bgp_aigp.py new file mode 100644 index 0000000..655e9ad --- /dev/null +++ b/tests/topotests/bgp_aigp/test_bgp_aigp.py @@ -0,0 +1,217 @@ +#!/usr/bin/env python +# SPDX-License-Identifier: ISC + +# +# Copyright (c) 2022 by +# Donatas Abraitis <donatas@opensourcerouting.org> +# + +""" +r7 sets aigp-metric for 10.0.0.71/32 to 71, and 72 for 10.0.0.72/32. + +r6 receives those routes with aigp-metric TLV. + +r2 and r3 receives those routes with aigp-metric TLV increased by 20, +and 30 appropriately. + +r1 receives routes with aigp-metric TLV 111,131 and 112,132 appropriately. +""" + +import os +import sys +import json +import pytest +import functools + +CWD = os.path.dirname(os.path.realpath(__file__)) +sys.path.append(os.path.join(CWD, "../")) + +# pylint: disable=C0413 +from lib import topotest +from lib.topogen import Topogen, TopoRouter, get_topogen +from lib.common_config import step + +pytestmark = [pytest.mark.bgpd] + + +def build_topo(tgen): + for routern in range(1, 8): + tgen.add_router("r{}".format(routern)) + + switch = tgen.add_switch("s1") + switch.add_link(tgen.gears["r1"]) + switch.add_link(tgen.gears["r2"]) + + switch = tgen.add_switch("s2") + switch.add_link(tgen.gears["r1"]) + switch.add_link(tgen.gears["r3"]) + + switch = tgen.add_switch("s3") + switch.add_link(tgen.gears["r2"]) + switch.add_link(tgen.gears["r4"]) + + switch = tgen.add_switch("s4") + switch.add_link(tgen.gears["r3"]) + switch.add_link(tgen.gears["r5"]) + + switch = tgen.add_switch("s5") + switch.add_link(tgen.gears["r4"]) + switch.add_link(tgen.gears["r6"]) + + switch = tgen.add_switch("s6") + switch.add_link(tgen.gears["r5"]) + switch.add_link(tgen.gears["r6"]) + + switch = tgen.add_switch("s7") + switch.add_link(tgen.gears["r6"]) + switch.add_link(tgen.gears["r7"]) + + +def setup_module(mod): + tgen = Topogen(build_topo, mod.__name__) + tgen.start_topology() + + router_list = tgen.routers() + + for i, (rname, router) in enumerate(router_list.items(), 1): + router.load_config( + TopoRouter.RD_ZEBRA, os.path.join(CWD, "{}/zebra.conf".format(rname)) + ) + router.load_config( + TopoRouter.RD_OSPF, os.path.join(CWD, "{}/ospfd.conf".format(rname)) + ) + router.load_config( + TopoRouter.RD_BGP, os.path.join(CWD, "{}/bgpd.conf".format(rname)) + ) + + tgen.start_router() + + +def teardown_module(mod): + tgen = get_topogen() + tgen.stop_topology() + + +def test_bgp_aigp(): + tgen = get_topogen() + + if tgen.routers_have_failure(): + pytest.skip(tgen.errors) + + r1 = tgen.gears["r1"] + 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.0.0.71/32 json")) + expected = { + "paths": [ + { + "aigpMetric": 111, + "valid": True, + "nexthops": [{"hostname": "r3", "accessible": True}], + }, + { + "aigpMetric": 131, + "valid": True, + "bestpath": {"selectionReason": "Neighbor IP"}, + "nexthops": [{"hostname": "r2", "accessible": True}], + }, + ] + } + return topotest.json_cmp(output, expected) + + def _bgp_check_aigp_metric(router, prefix, aigp): + output = json.loads( + router.vtysh_cmd("show bgp ipv4 unicast {} json".format(prefix)) + ) + expected = {"paths": [{"aigpMetric": aigp, "valid": True}]} + return topotest.json_cmp(output, expected) + + def _bgp_check_aigp_metric_bestpath(): + output = json.loads( + r1.vtysh_cmd( + "show bgp ipv4 unicast 10.0.0.64/28 longer-prefixes json detail" + ) + ) + expected = { + "routes": { + "10.0.0.71/32": { + "paths": [ + { + "aigpMetric": 111, + "bestpath": {"selectionReason": "AIGP"}, + "valid": True, + "nexthops": [{"hostname": "r3", "accessible": True}], + }, + { + "aigpMetric": 131, + "valid": True, + "nexthops": [{"hostname": "r2", "accessible": True}], + }, + ], + }, + "10.0.0.72/32": { + "paths": [ + { + "aigpMetric": 112, + "bestpath": {"selectionReason": "AIGP"}, + "valid": True, + "nexthops": [{"hostname": "r3", "accessible": True}], + }, + { + "aigpMetric": 132, + "valid": True, + "nexthops": [{"hostname": "r2", "accessible": True}], + }, + ], + }, + } + } + return topotest.json_cmp(output, expected) + + # Initial converge, AIGP is not involved in best-path selection process + test_func = functools.partial(_bgp_converge) + _, result = topotest.run_and_expect(test_func, None, count=60, wait=1) + assert result is None, "can't converge initially" + + # Enable `bgp bestpath aigp` + r1.vtysh_cmd( + """ + configure terminal + router bgp + bgp bestpath aigp + """ + ) + + # r4, 10.0.0.71/32 with aigp-metric 71 + test_func = functools.partial(_bgp_check_aigp_metric, r4, "10.0.0.71/32", 71) + _, 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 71" + + # r5, 10.0.0.72/32 with aigp-metric 72 + test_func = functools.partial(_bgp_check_aigp_metric, r5, "10.0.0.72/32", 72) + _, 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) + _, 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" + + # 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) + _, 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" + + # r1, check if AIGP is considered in best-path selection (lowest wins) + 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" + + +if __name__ == "__main__": + args = ["-s"] + sys.argv[1:] + sys.exit(pytest.main(args)) |