proxmox-mirrors/mirror_frr
1
0
Fork 0
mirror of https://git.proxmox.com/git/mirror_frr synced 2025-07-27 03:33:43 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

topotests: isis_srv6_topo1, fix wrong srte path

Browse Source 
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>
This commit is contained in:
Philippe Guibert 2025-01-22 17:18:03 +01:00
parent 2610c07831
commit 3a677993c0
1 changed files with 9 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
tests/topotests/isis_srv6_topo1/test_isis_srv6_topo1.py
View File

@ -333,7 +333,7 @@ def test_ping_step1():
# Setup encap route on rt1, decap route on rt2 # Setup encap route on rt1, decap route on rt2
tgen.gears["rt1"].vtysh_cmd( 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( tgen.gears["rt6"].vtysh_cmd(
"sharp install seg6local-routes fc00:0:f00d:: nexthop-seg6local eth-dst End_DT6 254 1" "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(): if tgen.routers_have_failure():
pytest.skip(tgen.errors) 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(): if tgen.routers_have_failure():
pytest.skip(tgen.errors) 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(): if tgen.routers_have_failure():
pytest.skip(tgen.errors) 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(): if tgen.routers_have_failure():
pytest.skip(tgen.errors) 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)
# #