The "2 HVs, 3 LS, 1 lport/LS, 2 peer LRs, static routes" test installs static routes to allow traffic to pass between peered routers properly. The second of the three static routes that is installed is targeted at router R2. However, the nexthop address on that static route is one of R2's own IP addresses. Also, the output port is a port on R1, not R2. The way the code works, the output port is seen as the source of truth, and even though the static route was targeted at R2, the actual logical flow for the static route is installed on R1 instead.
With this change, we change the static route to be installed on R1. This is because an upcoming change will limit our port search for static routes, and the test will fail without this change. Signed-off-by: Mark Michelson <[email protected]> --- tests/ovn.at | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/ovn.at b/tests/ovn.at index f1804a2ce..817d335f7 100644 --- a/tests/ovn.at +++ b/tests/ovn.at @@ -6663,7 +6663,7 @@ check ovn-nbctl lrp-add R2 R2_R1 00:00:00:02:03:05 20.0.0.2/24 peer=R1_R2 #install static routes check ovn-nbctl lr-route-add R1 172.16.1.0/24 20.0.0.2 -check ovn-nbctl lr-route-add R2 172.16.2.0/24 20.0.0.2 R1_R2 +check ovn-nbctl lr-route-add R1 172.16.2.0/24 20.0.0.2 R1_R2 check ovn-nbctl lr-route-add R2 192.168.1.0/24 20.0.0.1 # Create logical port foo1 in foo -- 2.55.0 _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
