topotests: isis_srv6_topo1, fix wrong srte path

There is no connectivity by using the proposed srv6 path.

> From Carmine:
> This seg6-route tells rt1 to steer packets destined to fc00:0:9::1 over this path: rt1->rt2->-rt6.
> This path is not correct. Since we are installing this seg6-route on rt1,
> it means that a packet matching this seg6-route has already reached rt1.
> So rt1 should not be part of the path.
> The correct path should be rt2->rt6.

Fix this by changing the proposed seg6 route. Also, the ping test should
be swapped, because invalidating the RT1 locator does not have any
impacts on the built SRv6 path.

Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

diff --git a/tests/topotests/isis_srv6_topo1/test_isis_srv6_topo1.py b/tests/topotests/isis_srv6_topo1/test_isis_srv6_topo1.py
index 035c60c9a3033fccd4bd3762ec9ab2726058f989..d17b4702f70848f571497e9e13bdf1e15d9648f1 100644 (file)
--- a/tests/topotests/isis_srv6_topo1/test_isis_srv6_topo1.py
+++ b/tests/topotests/isis_srv6_topo1/test_isis_srv6_topo1.py
@@ -333,7 +333,7 @@ def test_ping_step1():
 
     # Setup encap route on rt1, decap route on rt2
     tgen.gears["rt1"].vtysh_cmd(
-        "sharp install seg6-routes fc00:0:9::1 nexthop-seg6 2001:db8:1::2 encap fc00:0:1:2:6:f00d:: 1"
+        "sharp install seg6-routes fc00:0:9::1 nexthop-seg6 2001:db8:1::2 encap fc00:0:2:6:f00d:: 1"
     )
     tgen.gears["rt6"].vtysh_cmd(
         "sharp install seg6local-routes fc00:0:f00d:: nexthop-seg6local eth-dst End_DT6 254 1"
@@ -443,7 +443,8 @@ def test_ping_step2():
     if tgen.routers_have_failure():
         pytest.skip(tgen.errors)
 
-    check_ping6("rt1", "fc00:0:9::1", False)
+    # ping should pass because route to fc00:0:2:6:f00d:: is still valid
+    check_ping6("rt1", "fc00:0:9::1", True)
 
 
 #
@@ -643,7 +644,8 @@ def test_ping_step4():
     if tgen.routers_have_failure():
         pytest.skip(tgen.errors)
 
-    check_ping6("rt1", "fc00:0:9::1", False)
+    # ping should pass because route to fc00:0:2:6:f00d:: is still valid
+    check_ping6("rt1", "fc00:0:9::1", True)
 
 
 #
@@ -838,7 +840,8 @@ def test_ping_step6():
     if tgen.routers_have_failure():
         pytest.skip(tgen.errors)
 
-    check_ping6("rt1", "fc00:0:9::1", False)
+    # ping should pass because route to fc00:0:2:6:f00d:: is still valid
+    check_ping6("rt1", "fc00:0:9::1", True)
 
 
 #
@@ -1033,7 +1036,8 @@ def test_ping_step8():
     if tgen.routers_have_failure():
         pytest.skip(tgen.errors)
 
-    check_ping6("rt1", "fc00:0:9::1", False)
+    # ping should pass because route to fc00:0:2:6:f00d:: is still valid
+    check_ping6("rt1", "fc00:0:9::1", True)
 
 
 #