On Mon, Oct 19, 2026 at 10:52 PM Mark Michelson via dev < [email protected]> wrote:
> 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 49674266f..94ed06c63 100644 > --- a/tests/ovn.at > +++ b/tests/ovn.at > @@ -6662,7 +6662,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 > > Looks good to me, thanks. Acked-by: Ales Musil <[email protected]> _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
