tests: Check if connected routes are leaked between VRFs using accept-own

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

diff --git a/tests/topotests/bgp_accept_own/pe1/bgpd.conf b/tests/topotests/bgp_accept_own/pe1/bgpd.conf
index 8631293730ee48ccb8e0f5d29d751dd10e30bc49..109e0eadbb1dc103a2206824965d3fa2384fe5dd 100644 (file)
--- a/tests/topotests/bgp_accept_own/pe1/bgpd.conf
+++ b/tests/topotests/bgp_accept_own/pe1/bgpd.conf
@@ -24,6 +24,7 @@ router bgp 65001 vrf Customer
  neighbor 192.168.1.1 timers 1 3
  neighbor 192.168.1.1 timers connect 1
  address-family ipv4 unicast
+  redistribute connected
   label vpn export 10
   rd vpn export 192.168.1.2:2
   rt vpn import 192.168.1.2:2

diff --git a/tests/topotests/bgp_accept_own/pe1/zebra.conf b/tests/topotests/bgp_accept_own/pe1/zebra.conf
index 71476d2aefeaa249c38fc98f28cb35f5648377e2..2b7aefadf250c7544fc110db9ecef3e3d7b25213 100644 (file)
--- a/tests/topotests/bgp_accept_own/pe1/zebra.conf
+++ b/tests/topotests/bgp_accept_own/pe1/zebra.conf
@@ -11,5 +11,8 @@ interface pe1-eth1 vrf Service
 interface pe1-eth2
  ip address 10.0.1.1/24
 !
+interface pe1-eth3 vrf Customer
+ ip address 192.0.2.1/24
+!
 ip forwarding
 !

diff --git a/tests/topotests/bgp_accept_own/test_bgp_accept_own.py b/tests/topotests/bgp_accept_own/test_bgp_accept_own.py
index 161530b48390b0944d6fefe5dbd45710d06cc7b2..daef6dcd52c764c594ee63f5276f2b92dd02c018 100644 (file)
--- a/tests/topotests/bgp_accept_own/test_bgp_accept_own.py
+++ b/tests/topotests/bgp_accept_own/test_bgp_accept_own.py
@@ -58,6 +58,9 @@ def build_topo(tgen):
     switch.add_link(tgen.gears["pe1"])
     switch.add_link(tgen.gears["rr1"])
 
+    switch = tgen.add_switch("s4")
+    switch.add_link(tgen.gears["pe1"])
+
 
 def setup_module(mod):
     tgen = Topogen(build_topo, mod.__name__)
@@ -72,6 +75,7 @@ def setup_module(mod):
     pe1.run("ip link add Service type vrf table 1002")
     pe1.run("ip link set up dev Service")
     pe1.run("ip link set pe1-eth1 master Service")
+    pe1.run("ip link set pe1-eth3 master Customer")
     pe1.run("sysctl -w net.mpls.conf.pe1-eth2.input=1")
     rr1.run("sysctl -w net.mpls.conf.rr1-eth0.input=1")
 
@@ -112,7 +116,7 @@ def test_bgp_accept_own():
 
     def _bgp_check_received_routes_due_originator_id():
         output = json.loads(pe1.vtysh_cmd("show bgp ipv4 vpn summary json"))
-        expected = {"peers": {"10.10.10.101": {"pfxRcd": 0, "pfxSnt": 4}}}
+        expected = {"peers": {"10.10.10.101": {"pfxRcd": 0, "pfxSnt": 5}}}
         return topotest.json_cmp(output, expected)
 
     test_func = functools.partial(_bgp_check_received_routes_due_originator_id)
@@ -134,7 +138,7 @@ def test_bgp_accept_own():
 
     def _bgp_check_received_routes_with_modified_rts():
         output = json.loads(pe1.vtysh_cmd("show bgp ipv4 vpn summary json"))
-        expected = {"peers": {"10.10.10.101": {"pfxRcd": 4, "pfxSnt": 4}}}
+        expected = {"peers": {"10.10.10.101": {"pfxRcd": 5, "pfxSnt": 5}}}
         return topotest.json_cmp(output, expected)
 
     test_func = functools.partial(_bgp_check_received_routes_with_modified_rts)
@@ -154,9 +158,7 @@ def test_bgp_accept_own():
         expected = {
             "paths": [
                 {
-                    "community": {
-                        "string": "65001:111"
-                    },
+                    "community": {"string": "65001:111"},
                     "extendedCommunity": {
                         "string": "RT:192.168.1.2:2 RT:192.168.2.2:2"
                     },
@@ -171,6 +173,37 @@ def test_bgp_accept_own():
         result is None
     ), "Failed, routes are not imported from RR1 with modified RT list"
 
+    step("Check if 192.0.2.0/24 is imported to vrf Service from vrf Customer")
+
+    def _bgp_check_imported_local_routes_from_vrf_service():
+        output = json.loads(
+            pe1.vtysh_cmd("show ip route vrf Service 192.0.2.0/24 json")
+        )
+        expected = {
+            "192.0.2.0/24": [
+                {
+                    "vrfName": "Service",
+                    "table": 1002,
+                    "installed": True,
+                    "selected": True,
+                    "nexthops": [
+                        {
+                            "fib": True,
+                            "vrf": "Customer",
+                            "active": True,
+                        }
+                    ],
+                }
+            ]
+        }
+        return topotest.json_cmp(output, expected)
+
+    test_func = functools.partial(_bgp_check_imported_local_routes_from_vrf_service)
+    _, result = topotest.run_and_expect(test_func, None, count=60, wait=1)
+    assert (
+        result is None
+    ), "Failed, didn't import local route 192.0.2.0/24 from vrf Customer to vrf Service"
+
     step("Check if 172.16.255.1/32 is announced to CE2")
 
     def _bgp_check_received_routes_from_pe():
@@ -188,9 +221,7 @@ def test_bgp_accept_own():
 
     test_func = functools.partial(_bgp_check_received_routes_from_pe)
     _, result = topotest.run_and_expect(test_func, None, count=60, wait=1)
-    assert (
-        result is None
-    ), "Failed, didn't receive 172.16.255.1/32 from PE1"
+    assert result is None, "Failed, didn't receive 172.16.255.1/32 from PE1"
 
 
 if __name__ == "__main__":