diff options
| author | Donatas Abraitis <donatas@opensourcerouting.org> | 2024-11-07 14:01:39 +0200 | 
|---|---|---|
| committer | Mergify <37929162+mergify[bot]@users.noreply.github.com> | 2024-11-07 16:07:12 +0000 | 
| commit | 0b1bb842421723e183d9cd36b794d2dd9a2aed0d (patch) | |
| tree | cb788d2474b679569accd6e93e5250ce9421fa5a | |
| parent | 86532b3dc25e654d17a9e236a972278432b617ea (diff) | |
tests: Check if stale routes with addpath are marked with LLGR community
Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
(cherry picked from commit 2e5e3b4bd097dc5b45dee476d844bde398ebee00)
| -rw-r--r-- | tests/topotests/bgp_addpath_llgr/__init__.py | 0 | ||||
| -rw-r--r-- | tests/topotests/bgp_addpath_llgr/r1/frr.conf | 23 | ||||
| -rw-r--r-- | tests/topotests/bgp_addpath_llgr/r2/frr.conf | 29 | ||||
| -rw-r--r-- | tests/topotests/bgp_addpath_llgr/r3/frr.conf | 14 | ||||
| -rw-r--r-- | tests/topotests/bgp_addpath_llgr/test_bgp_addpath_llgr.py | 131 | 
5 files changed, 197 insertions, 0 deletions
diff --git a/tests/topotests/bgp_addpath_llgr/__init__.py b/tests/topotests/bgp_addpath_llgr/__init__.py new file mode 100644 index 0000000000..e69de29bb2 --- /dev/null +++ b/tests/topotests/bgp_addpath_llgr/__init__.py diff --git a/tests/topotests/bgp_addpath_llgr/r1/frr.conf b/tests/topotests/bgp_addpath_llgr/r1/frr.conf new file mode 100644 index 0000000000..0644cf63a8 --- /dev/null +++ b/tests/topotests/bgp_addpath_llgr/r1/frr.conf @@ -0,0 +1,23 @@ +! +int r1-eth0 + ip address 192.168.1.1/24 +! +int r1-eth1 + ip address 192.168.2.1/24 +! +router bgp 65001 + no bgp ebgp-requires-policy + no bgp network import-check + neighbor 192.168.1.2 remote-as auto + neighbor 192.168.1.2 timers 1 3 + neighbor 192.168.1.2 timers connect 1 + neighbor 192.168.2.2 remote-as auto + neighbor 192.168.2.2 timers 1 3 + neighbor 192.168.2.2 timers connect 1 + neighbor r1-eth1 interface remote-as auto + neighbor r1-eth1 timers 1 3 + neighbor r1-eth1 timers connect 1 + address-family ipv4 unicast +  network 10.0.0.1/32 + exit-address-family +! diff --git a/tests/topotests/bgp_addpath_llgr/r2/frr.conf b/tests/topotests/bgp_addpath_llgr/r2/frr.conf new file mode 100644 index 0000000000..7f6fff0fda --- /dev/null +++ b/tests/topotests/bgp_addpath_llgr/r2/frr.conf @@ -0,0 +1,29 @@ +! +int r2-eth0 + ip address 192.168.1.2/24 +! +int r2-eth1 + ip address 192.168.2.2/24 +! +int r2-eth2 + ip address 192.168.3.2/24 +! +router bgp 65002 + bgp graceful-restart + bgp graceful-restart preserve-fw-state + bgp graceful-restart restart-time 0 + bgp long-lived-graceful-restart stale-time 10 + no bgp ebgp-requires-policy + neighbor 192.168.1.1 remote-as auto + neighbor 192.168.1.1 timers 1 3 + neighbor 192.168.1.1 timers connect 1 + neighbor 192.168.2.1 remote-as auto + neighbor 192.168.2.1 timers 1 3 + neighbor 192.168.2.1 timers connect 1 + neighbor 192.168.3.3 remote-as auto + neighbor 192.168.3.3 timers 1 3 + neighbor 192.168.3.3 timers connect 1 + address-family ipv4 unicast +  neighbor 192.168.3.3 addpath-tx-all-paths + exit-address-family +! diff --git a/tests/topotests/bgp_addpath_llgr/r3/frr.conf b/tests/topotests/bgp_addpath_llgr/r3/frr.conf new file mode 100644 index 0000000000..f36492d7e0 --- /dev/null +++ b/tests/topotests/bgp_addpath_llgr/r3/frr.conf @@ -0,0 +1,14 @@ +! +int r3-eth0 + ip address 192.168.3.3/24 +! +router bgp 65003 + bgp graceful-restart + bgp graceful-restart preserve-fw-state + bgp graceful-restart restart-time 0 + bgp long-lived-graceful-restart stale-time 10 + no bgp ebgp-requires-policy + neighbor 192.168.3.2 remote-as auto + neighbor 192.168.3.2 timers 1 3 + neighbor 192.168.3.2 timers connect 1 +! diff --git a/tests/topotests/bgp_addpath_llgr/test_bgp_addpath_llgr.py b/tests/topotests/bgp_addpath_llgr/test_bgp_addpath_llgr.py new file mode 100644 index 0000000000..fa0314304b --- /dev/null +++ b/tests/topotests/bgp_addpath_llgr/test_bgp_addpath_llgr.py @@ -0,0 +1,131 @@ +#!/usr/bin/env python +# SPDX-License-Identifier: ISC + +# Copyright (c) 2024 by +# Donatas Abraitis <donatas@opensourcerouting.org> +# + +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, get_topogen +from lib.common_config import kill_router_daemons + +pytestmark = [pytest.mark.bgpd] + + +def setup_module(mod): +    topodef = {"s1": ("r1", "r2"), "s2": ("r1", "r2"), "s3": ("r2", "r3")} +    tgen = Topogen(topodef, mod.__name__) +    tgen.start_topology() + +    router_list = tgen.routers() + +    for _, (rname, router) in enumerate(router_list.items(), 1): +        router.load_frr_config(os.path.join(CWD, "{}/frr.conf".format(rname))) + +    tgen.start_router() + + +def teardown_module(mod): +    tgen = get_topogen() +    tgen.stop_topology() + + +def test_bgp_addpath_llgr(): +    tgen = get_topogen() + +    if tgen.routers_have_failure(): +        pytest.skip(tgen.errors) + +    r2 = tgen.gears["r2"] +    r3 = tgen.gears["r3"] + +    def _bgp_converge(): +        output = json.loads(r2.vtysh_cmd("show bgp ipv4 unicast summary json")) +        expected = { +            "peers": { +                "192.168.1.1": { +                    "hostname": "r1", +                    "remoteAs": 65001, +                    "localAs": 65002, +                    "pfxRcd": 1, +                    "state": "Established", +                }, +                "192.168.2.1": { +                    "hostname": "r1", +                    "remoteAs": 65001, +                    "localAs": 65002, +                    "pfxRcd": 1, +                    "state": "Established", +                }, +                "192.168.3.3": { +                    "hostname": "r3", +                    "remoteAs": 65003, +                    "localAs": 65002, +                    "pfxSnt": 2, +                    "state": "Established", +                }, +            } +        } +        return topotest.json_cmp(output, expected) + +    test_func = functools.partial( +        _bgp_converge, +    ) +    _, result = topotest.run_and_expect(test_func, None, count=30, wait=1) +    assert result is None, "Initial peering failed" + +    kill_router_daemons(tgen, "r2", ["bgpd"]) + +    def _bgp_check_stale_llgr_routes(): +        output = json.loads(r3.vtysh_cmd("show bgp ipv4 unicast 10.0.0.1/32 json")) +        expected = { +            "paths": [ +                { +                    "stale": True, +                    "valid": True, +                    "community": {"string": "llgr-stale", "list": ["llgrStale"]}, +                }, +                { +                    "stale": True, +                    "valid": True, +                    "community": {"string": "llgr-stale", "list": ["llgrStale"]}, +                }, +            ] +        } +        return topotest.json_cmp(output, expected) + +    test_func = functools.partial( +        _bgp_check_stale_llgr_routes, +    ) +    _, result = topotest.run_and_expect(test_func, None, count=30, wait=1) +    assert result is None, "Can't see stale LLGR routes" + +    def _bgp_check_stale_routes_cleared(): +        output = json.loads(r3.vtysh_cmd("show bgp ipv4 unicast json")) +        expected = { +            "routes": { +                "10.0.0.1/32": None, +            } +        } +        return topotest.json_cmp(output, expected) + +    test_func = functools.partial( +        _bgp_check_stale_routes_cleared, +    ) +    _, result = topotest.run_and_expect(test_func, None, count=30, wait=1) +    assert result is None, "Can't see stale routes" + + +if __name__ == "__main__": +    args = ["-s"] + sys.argv[1:] +    sys.exit(pytest.main(args))  | 
