On Thu, Nov 13, 2025 at 9:55 AM Dumitru Ceara <[email protected]> wrote:

> On 11/12/25 10:05 AM, Ales Musil wrote:
> > The dynamic-routing test was rebinding virtual port almost back to
> > back. That could lead to a situation when the virtual port wouldn't
> > be bound because of the backoff. Make sure the backoff is removed
> > before we attempt to bind it again.
> >
> > Fixes: 0bf8aec8c178 ("northd: Fix virtual port ip advertisment via BGP.")
> > Signed-off-by: Ales Musil <[email protected]>
> > ---
>
> Hi Ales,
>
> This looks good to me, thanks for the fix!
>
> Acked-by: Dumitru Ceara <[email protected]>
>
> The CI failure seems to be an infra issue:
>
> Run sudo podman load -i /tmp/image.tar
> Error: stat /tmp/image.tar: no such file or directory
> Error: Process completed with exit code 125.
>
> Recheck-request: github-robot-_Build_and_Test
>
> >  tests/system-ovn.at | 4 ++++
> >  1 file changed, 4 insertions(+)
> >
> > diff --git a/tests/system-ovn.at b/tests/system-ovn.at
> > index 5b34e621f..dbf3751ec 100644
> > --- a/tests/system-ovn.at
> > +++ b/tests/system-ovn.at
> > @@ -16209,6 +16209,8 @@ blackhole 198.51.100.0/24 proto ovn metric 1000
> >  233.252.0.0/24 via 192.168.10.10 dev lo onlink
> >  233.253.0.0/24 via 192.168.20.20 dev hv1-mll onlink])
> >
> > +# Remove the backoff period, so we can bind it right away.
> > +check ovn-sbctl remove Port_Binding vip options vport-backoff
> >  # Rebind "vip" locally.
> >  NS_EXEC([vif4], [arping -U -c 1 -w 2 -I vif4 192.0.2.30])
> >  wait_column "vif4" Port_Binding virtual_parent logical_port=vip
> > @@ -16256,6 +16258,8 @@ blackhole 198.51.100.0/24 proto ovn metric 1000
> >  233.252.0.0/24 via 192.168.10.10 dev lo onlink
> >  233.253.0.0/24 via 192.168.20.20 dev hv1-mll onlink])
> >
> > +# Remove the backoff period, so we can bind it right away.
> > +check ovn-sbctl remove Port_Binding vip options vport-backoff
> >  # Rebind "vip" locally.
> >  NS_EXEC([vif4], [arping -U -c 1 -w 2 -I vif4 192.0.2.30])
> >  wait_column "vif4" Port_Binding virtual_parent logical_port=vip
>
> Regards,
> Dumitru
>
>
Thank you Dumitru,

I went ahead, merged this into main and backported down to 25.03.

Regards,
Ales
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to