From: Louis Scalbert <louis.scalbert@6wind.com>
Date: Thu, 24 Apr 2025 09:06:10 +0000 (+0200)
Subject: tests: bgp_evpn_rt5 test match evpn vni
X-Git-Url: https://git.puffer.fish/?a=commitdiff_plain;h=refs%2Fpull%2F18708%2Fhead;p=mirror%2Ffrr.git

tests: bgp_evpn_rt5 test match evpn vni

0d5bd461af commit was supposed to check that route-map "match evpn vni"
was working by adding a r3 router that injected prefixes with VNI 102.
A route-map on r1 was filtering r3 prefixes. However, the test was
working even after removing r3. A check was needed without the route-map
to check that r3 prefixes were received before applying the filter.

Test "match evpn vni" route-map filtering applied to r2 input instead.

Fixes: 0d5bd461af ("topotests: bgp_evpn_rt5, add test with match evpn vni command")
Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com>
---

diff --git a/tests/topotests/bgp_evpn_rt5/test_bgp_evpn.py b/tests/topotests/bgp_evpn_rt5/test_bgp_evpn.py
index cf42f1f4a1..c5189a1cec 100644
--- a/tests/topotests/bgp_evpn_rt5/test_bgp_evpn.py
+++ b/tests/topotests/bgp_evpn_rt5/test_bgp_evpn.py
@@ -754,6 +754,157 @@ def test_shutdown_multipath_check_next_hops():
     _test_router_check_evpn_next_hop()
 
 
+def test_rmap_match_evpn_vni_102():
+    """
+    change input route-map from r2.
+    match evpn vni value from 101 to 102
+    expecting all prefixes are denied
+    """
+
+    tgen = get_topogen()
+    if tgen.routers_have_failure():
+        pytest.skip(tgen.errors)
+
+    r1 = tgen.gears["r1"]
+    nb_prefix = 2
+    expected = {"numPrefix": nb_prefix, "totalPrefix": nb_prefix}
+    test_func = partial(
+        topotest.router_json_cmp,
+        r1,
+        "show bgp l2vpn evpn rd 65000:2 json",
+        expected,
+    )
+    _, result = topotest.run_and_expect(test_func, None, count=20, wait=1)
+    assert result is None, f"r1 was expecting {nb_prefix} from r2"
+
+    # change route-map and test
+    cfg = {
+        "r1": {
+            "raw_config": [
+                "route-map rmap_r1 permit 1",
+                "match evpn vni 102",
+            ]
+        },
+    }
+    assert apply_raw_config(tgen, cfg), "Configuration failed"
+
+    nb_prefix = 0
+    expected = {"numPrefix": nb_prefix, "totalPrefix": nb_prefix}
+    test_func = partial(
+        topotest.router_json_cmp,
+        r1,
+        "show bgp l2vpn evpn rd 65000:2 json",
+        expected,
+    )
+    _, result = topotest.run_and_expect(test_func, None, count=20, wait=1)
+    assert result is None, f"r1 was expecting {nb_prefix} from r2"
+
+
+def test_rmap_match_evpn_vni_101():
+    """
+    change input route-map from r2.
+    re-apply match evpn vni value 101
+    expecting all prefixes are received
+    """
+
+    tgen = get_topogen()
+    if tgen.routers_have_failure():
+        pytest.skip(tgen.errors)
+
+    # change route-map and test
+    cfg = {
+        "r1": {
+            "raw_config": [
+                "route-map rmap_r1 permit 1",
+                "match evpn vni 101",
+            ]
+        },
+    }
+    assert apply_raw_config(tgen, cfg), "Configuration failed"
+
+    r1 = tgen.gears["r1"]
+    nb_prefix = 2
+    expected = {"numPrefix": nb_prefix, "totalPrefix": nb_prefix}
+    test_func = partial(
+        topotest.router_json_cmp,
+        r1,
+        "show bgp l2vpn evpn rd 65000:2 json",
+        expected,
+    )
+    _, result = topotest.run_and_expect(test_func, None, count=20, wait=1)
+    assert result is None, f"r1 was expecting {nb_prefix} from r2"
+
+
+def test_rmap_match_evpn_vni_101_deny():
+    """
+    change input route-map from r2.
+    set deny action to vni 101
+    expecting all prefixes are denied
+    """
+
+    tgen = get_topogen()
+    if tgen.routers_have_failure():
+        pytest.skip(tgen.errors)
+
+    # change route-map and test
+    cfg = {
+        "r1": {
+            "raw_config": [
+                "route-map rmap_r1 deny 1",
+            ]
+        },
+    }
+    assert apply_raw_config(tgen, cfg), "Configuration failed"
+
+    r1 = tgen.gears["r1"]
+    nb_prefix = 0
+    expected = {"numPrefix": nb_prefix, "totalPrefix": nb_prefix}
+    test_func = partial(
+        topotest.router_json_cmp,
+        r1,
+        "show bgp l2vpn evpn rd 65000:2 json",
+        expected,
+    )
+    _, result = topotest.run_and_expect(test_func, None, count=20, wait=1)
+    assert result is None, f"r1 was expecting {nb_prefix} from r2"
+
+
+def test_no_rmap_match_evpn_vni():
+    """
+    un-apply input route-map from r2
+    expecting all prefixes are received
+    """
+
+    tgen = get_topogen()
+    if tgen.routers_have_failure():
+        pytest.skip(tgen.errors)
+
+    # change route-map and test
+    cfg = {
+        "r1": {
+            "raw_config": [
+                "router bgp 65000",
+                " address-family l2vpn evpn",
+                "  no neighbor 192.168.0.2 route-map rmap_r1 in",
+                "  no neighbor 192.168.99.2 route-map rmap_r1 in",
+            ]
+        },
+    }
+    assert apply_raw_config(tgen, cfg), "Configuration failed"
+
+    r1 = tgen.gears["r1"]
+    nb_prefix = 2
+    expected = {"numPrefix": nb_prefix, "totalPrefix": nb_prefix}
+    test_func = partial(
+        topotest.router_json_cmp,
+        r1,
+        "show bgp l2vpn evpn rd 65000:2 json",
+        expected,
+    )
+    _, result = topotest.run_and_expect(test_func, None, count=30, wait=1)
+    assert result is None, f"r1 was expecting {nb_prefix} from r2"
+
+
 def test_memory_leak():
     "Run the memory leak test and report results."
     tgen = get_topogen()