From 1d957df8801ac006f3b6cc545b13f2c8e9d99c70 Mon Sep 17 00:00:00 2001 From: Tuetuopay Date: Fri, 14 Mar 2025 20:21:53 +0100 Subject: [PATCH] tests: add route-map evpn set gateway-ip topotest This test does not actually look at the route since the gateway-ip is not exposed in vtysh output. However, this ensures such a route-map does not crash bgpd. Signed-off-by: Tuetuopay (cherry picked from commit 05a74323b9d440590074f22be11c34a605da69ad) --- .../bgp_evpn_route_map_set/__init__.py | 0 .../bgp_evpn_route_map_set/c1/frr.conf | 22 +++++ .../bgp_evpn_route_map_set/r1/frr.conf | 38 ++++++++ .../bgp_evpn_route_map_set/r2/frr.conf | 21 +++++ .../test_bgp_evpn_route_map_set.py | 88 +++++++++++++++++++ 5 files changed, 169 insertions(+) create mode 100644 tests/topotests/bgp_evpn_route_map_set/__init__.py create mode 100644 tests/topotests/bgp_evpn_route_map_set/c1/frr.conf create mode 100644 tests/topotests/bgp_evpn_route_map_set/r1/frr.conf create mode 100644 tests/topotests/bgp_evpn_route_map_set/r2/frr.conf create mode 100644 tests/topotests/bgp_evpn_route_map_set/test_bgp_evpn_route_map_set.py diff --git a/tests/topotests/bgp_evpn_route_map_set/__init__.py b/tests/topotests/bgp_evpn_route_map_set/__init__.py new file mode 100644 index 0000000000..e69de29bb2 diff --git a/tests/topotests/bgp_evpn_route_map_set/c1/frr.conf b/tests/topotests/bgp_evpn_route_map_set/c1/frr.conf new file mode 100644 index 0000000000..05c714e3a4 --- /dev/null +++ b/tests/topotests/bgp_evpn_route_map_set/c1/frr.conf @@ -0,0 +1,22 @@ +int lo + ip address 10.10.10.10/32 +! +int c1-eth0 + ip address 10.0.0.1/31 +! +router bgp 65000 + no bgp ebgp-requires-policy + no bgp network import-check + ! + neighbor 10.0.0.0 remote-as external + neighbor 10.0.0.0 timers 1 3 + neighbor 10.0.0.0 timers connect 1 + ! + address-family ipv4 unicast + neighbor 10.0.0.0 activate + redistribute connected route-map allow-lo + exit-address-family +! +route-map allow-lo permit 10 + match interface lo +! diff --git a/tests/topotests/bgp_evpn_route_map_set/r1/frr.conf b/tests/topotests/bgp_evpn_route_map_set/r1/frr.conf new file mode 100644 index 0000000000..3630bdb980 --- /dev/null +++ b/tests/topotests/bgp_evpn_route_map_set/r1/frr.conf @@ -0,0 +1,38 @@ +! This router advertises as EVPN type-5 routes those received in a plain +! ipv4 unicast session +! +vni 10 +! +int r1-eth0 + ip address 10.0.0.0/31 +! +int r1-eth1 + ip address 10.0.0.2/31 +! +router bgp 64000 + no bgp ebgp-requires-policy + no bgp network import-check + ! + neighbor 10.0.0.1 remote-as external + neighbor 10.0.0.1 timers 1 3 + neighbor 10.0.0.1 timers connect 1 + ! + address-family ipv4 unicast + neighbor 10.0.0.1 activate + ! + neighbor 10.0.0.3 remote-as internal + neighbor 10.0.0.3 timers 1 3 + neighbor 10.0.0.3 timers connect 1 + ! + address-family l2vpn evpn + neighbor 10.0.0.3 activate + neighbor 10.0.0.3 route-map set-gw-ip out + ! + advertise-all-vni + ! + advertise ipv4 unicast + exit-address-family +! +route-map set-gw-ip permit 10 + set evpn gateway-ip ipv4 10.10.10.10 +! diff --git a/tests/topotests/bgp_evpn_route_map_set/r2/frr.conf b/tests/topotests/bgp_evpn_route_map_set/r2/frr.conf new file mode 100644 index 0000000000..754b696bfd --- /dev/null +++ b/tests/topotests/bgp_evpn_route_map_set/r2/frr.conf @@ -0,0 +1,21 @@ +vni 10 +! +int r2-eth0 + ip address 10.0.0.3/31 +! +router bgp 64000 + no bgp default ipv4-unicast + ! + neighbor 10.0.0.2 remote-as internal + neighbor 10.0.0.2 timers 1 3 + neighbor 10.0.0.2 timers connect 1 + ! + address-family l2vpn evpn + neighbor 10.0.0.2 activate + ! workaround for now, as frr drops some evpn attributes like gw-ip if soft + ! reconf is not enabled for inbound routes + neighbor 10.0.0.2 soft-reconfiguration inbound + ! + advertise-all-vni + exit-address-family +! diff --git a/tests/topotests/bgp_evpn_route_map_set/test_bgp_evpn_route_map_set.py b/tests/topotests/bgp_evpn_route_map_set/test_bgp_evpn_route_map_set.py new file mode 100644 index 0000000000..eb00d6db69 --- /dev/null +++ b/tests/topotests/bgp_evpn_route_map_set/test_bgp_evpn_route_map_set.py @@ -0,0 +1,88 @@ +#!/usr/bin/env python +# SPDX-License-Identifier: GPL-2.0-or-later + +# Copyright (c) 2025 Tuetuopay + +""" +Test if route-map set for EVPN fields works. +""" + +import os +import sys +import json +import pytest +import functools + +pytestmark = [pytest.mark.bgpd] + +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 TopoRouter, Topogen, get_topogen +from lib.topolog import logger + +def setup_module(mod): + topodef = {"s1": ("c1", "r1"), "s2": ("r1", "r2")} + tgen = Topogen(topodef, mod.__name__) + tgen.start_topology() + + tgen.net["r1"].cmd( + """ +ip link add br10 up type bridge +ip link add vxlan10 up master br10 type vxlan id 10 dstport 4789 local 10.0.0.2 nolearning + """ + ) + tgen.net["r2"].cmd( + """ +ip link add br10 up type bridge +ip link add vxlan10 up master br10 type vxlan id 10 dstport 4789 local 10.0.0.3 nolearning + """ + ) + + for name, router in tgen.routers().items(): + router.load_frr_config(os.path.join(CWD, f"{name}/frr.conf")) + + tgen.start_router() + + +def teardown_module(mod): + get_topogen().stop_topology() + + +def test_bgp_evpn_route_map_set_gateway_ip(): + tgen: Topogen = get_topogen() + + if tgen.routers_have_failure(): + pytest.skip(tgen.errors) + + r2: TopoRouter = tgen.gears["r2"] + def _bgp_converge(): + output = json.loads(r2.vtysh_cmd("show bgp l2vpn evpn all overlay json")) + expected = { + "10.0.0.2:1": { + "[5]:[0]:[32]:[10.10.10.10]": { + "paths": [ + { + "valid": True, + "overlay": { + "gw": "10.10.10.10", + }, + }, + ], + }, + }, + "numPrefix": 1, + } + return topotest.json_cmp(output, expected) + + logger.info("Check route type-5 gateway-ip") + test_func = functools.partial(_bgp_converge) + _, result = topotest.run_and_expect(test_func, None, count=60, wait=1) + assert result is None, "EVPN Route with gateway-ip should be advertised" + + +if __name__ == "__main__": + args = ["-s"] + sys.argv[1:] + sys.exit(pytest.main(args)) -- 2.39.5