From 8cfca51df5891bd797470677977d69156c9321e0 Mon Sep 17 00:00:00 2001 From: Donatas Abraitis Date: Tue, 22 Jun 2021 16:44:30 +0300 Subject: [PATCH] tests: Add an additional conditional BGP default route advertisement Signed-off-by: Donatas Abraitis --- .../__init__.py | 0 .../r1/bgpd.conf | 13 ++ .../r1/zebra.conf | 6 + .../r2/bgpd.conf | 8 + .../r2/zebra.conf | 9 ++ ..._bgp_default-originate_route-map_match2.py | 138 ++++++++++++++++++ 6 files changed, 174 insertions(+) create mode 100644 tests/topotests/bgp_default_route_route_map_match2/__init__.py create mode 100644 tests/topotests/bgp_default_route_route_map_match2/r1/bgpd.conf create mode 100644 tests/topotests/bgp_default_route_route_map_match2/r1/zebra.conf create mode 100644 tests/topotests/bgp_default_route_route_map_match2/r2/bgpd.conf create mode 100644 tests/topotests/bgp_default_route_route_map_match2/r2/zebra.conf create mode 100644 tests/topotests/bgp_default_route_route_map_match2/test_bgp_default-originate_route-map_match2.py diff --git a/tests/topotests/bgp_default_route_route_map_match2/__init__.py b/tests/topotests/bgp_default_route_route_map_match2/__init__.py new file mode 100644 index 0000000000..e69de29bb2 diff --git a/tests/topotests/bgp_default_route_route_map_match2/r1/bgpd.conf b/tests/topotests/bgp_default_route_route_map_match2/r1/bgpd.conf new file mode 100644 index 0000000000..ee7a92f116 --- /dev/null +++ b/tests/topotests/bgp_default_route_route_map_match2/r1/bgpd.conf @@ -0,0 +1,13 @@ +router bgp 65000 + no bgp ebgp-requires-policy + neighbor 192.168.255.2 remote-as 65001 + neighbor 192.168.255.2 timers 3 10 + address-family ipv4 unicast + neighbor 192.168.255.2 default-originate route-map default + exit-address-family +! +ip prefix-list r2 permit 10.0.0.0/22 +! +route-map default permit 10 + match ip address prefix-list r2 +! diff --git a/tests/topotests/bgp_default_route_route_map_match2/r1/zebra.conf b/tests/topotests/bgp_default_route_route_map_match2/r1/zebra.conf new file mode 100644 index 0000000000..e2c399e536 --- /dev/null +++ b/tests/topotests/bgp_default_route_route_map_match2/r1/zebra.conf @@ -0,0 +1,6 @@ +! +interface r1-eth0 + ip address 192.168.255.1/24 +! +ip forwarding +! diff --git a/tests/topotests/bgp_default_route_route_map_match2/r2/bgpd.conf b/tests/topotests/bgp_default_route_route_map_match2/r2/bgpd.conf new file mode 100644 index 0000000000..00c96cc58b --- /dev/null +++ b/tests/topotests/bgp_default_route_route_map_match2/r2/bgpd.conf @@ -0,0 +1,8 @@ +router bgp 65001 + no bgp ebgp-requires-policy + neighbor 192.168.255.1 remote-as 65000 + neighbor 192.168.255.1 timers 3 10 + address-family ipv4 unicast + redistribute connected + exit-address-family +! diff --git a/tests/topotests/bgp_default_route_route_map_match2/r2/zebra.conf b/tests/topotests/bgp_default_route_route_map_match2/r2/zebra.conf new file mode 100644 index 0000000000..f355ab1517 --- /dev/null +++ b/tests/topotests/bgp_default_route_route_map_match2/r2/zebra.conf @@ -0,0 +1,9 @@ +! +interface lo + ip address 10.0.0.1/22 +! +interface r2-eth0 + ip address 192.168.255.2/24 +! +ip forwarding +! diff --git a/tests/topotests/bgp_default_route_route_map_match2/test_bgp_default-originate_route-map_match2.py b/tests/topotests/bgp_default_route_route_map_match2/test_bgp_default-originate_route-map_match2.py new file mode 100644 index 0000000000..42a6b6edf6 --- /dev/null +++ b/tests/topotests/bgp_default_route_route_map_match2/test_bgp_default-originate_route-map_match2.py @@ -0,0 +1,138 @@ +#!/usr/bin/env python + +# Copyright (c) 2021 by +# Donatas Abraitis +# +# Permission to use, copy, modify, and/or distribute this software +# for any purpose with or without fee is hereby granted, provided +# that the above copyright notice and this permission notice appear +# in all copies. +# +# THE SOFTWARE IS PROVIDED "AS IS" AND NETDEF DISCLAIMS ALL WARRANTIES +# WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF +# MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL NETDEF BE LIABLE FOR +# ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY +# DAMAGES WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, +# WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS +# ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR PERFORMANCE +# OF THIS SOFTWARE. +# + +""" +Test if default-originate works with conditional match. +If 10.0.0.0/22 is recived from r2, then we announce 0.0.0.0/0 +to r2. +""" + +import os +import sys +import json +import time +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.topolog import logger +from mininet.topo import Topo +from lib.common_config import step + + +class TemplateTopo(Topo): + def build(self, *_args, **_opts): + tgen = get_topogen(self) + + for routern in range(1, 3): + tgen.add_router("r{}".format(routern)) + + switch = tgen.add_switch("s1") + switch.add_link(tgen.gears["r1"]) + switch.add_link(tgen.gears["r2"]) + + +def setup_module(mod): + tgen = Topogen(TemplateTopo, 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_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_default_originate_route_map(): + tgen = get_topogen() + + if tgen.routers_have_failure(): + pytest.skip(tgen.errors) + + router = tgen.gears["r2"] + + def _bgp_converge(router): + output = json.loads(router.vtysh_cmd("show ip bgp neighbor 192.168.255.1 json")) + expected = { + "192.168.255.1": { + "bgpState": "Established", + "addressFamilyInfo": {"ipv4Unicast": {"acceptedPrefixCounter": 1}}, + } + } + return topotest.json_cmp(output, expected) + + def _bgp_default_route_is_valid(router): + output = json.loads(router.vtysh_cmd("show ip bgp 0.0.0.0/0 json")) + expected = {"paths": [{"valid": True}]} + return topotest.json_cmp(output, expected) + + step("Converge network") + test_func = functools.partial(_bgp_converge, router) + success, result = topotest.run_and_expect(test_func, None, count=30, wait=0.5) + assert result is None, "Failed to see bgp convergence at r2" + + step("Withdraw 10.0.0.0/22 from R2") + router.vtysh_cmd( + "conf t\nrouter bgp\naddress-family ipv4\nno redistribute connected" + ) + + step("Check if we don't have 0.0.0.0/0 at R2") + test_func = functools.partial(_bgp_default_route_is_valid, router) + success, result = topotest.run_and_expect(test_func, not None, count=30, wait=0.5) + assert result is not None, "0.0.0.0/0 exists at r2" + + step("Announce 10.0.0.0/22 from R2") + router.vtysh_cmd("conf t\nrouter bgp\naddress-family ipv4\nredistribute connected") + + step("Check if we have 0.0.0.0/0 at R2") + test_func = functools.partial(_bgp_default_route_is_valid, router) + success, result = topotest.run_and_expect(test_func, None, count=30, wait=0.5) + assert result is None, "0.0.0.0/0 does not exist at r2" + + step("Withdraw 10.0.0.0/22 from R2 again") + router.vtysh_cmd( + "conf t\nrouter bgp\naddress-family ipv4\nno redistribute connected" + ) + + step("Check if we don't have 0.0.0.0/0 at R2 again") + test_func = functools.partial(_bgp_default_route_is_valid, router) + success, result = topotest.run_and_expect(test_func, not None, count=30, wait=0.5) + assert result is not None, "0.0.0.0/0 exists at r2" + + +if __name__ == "__main__": + args = ["-s"] + sys.argv[1:] + sys.exit(pytest.main(args))