On 7/7/26 3:53 PM, Xavier Simonart via dev wrote:
> ovn-nbctl --wait=hv does not wait for tunnel creation to be completed:
> ovn creates the interfaces in ovs db, then ovs-vswitchd writes ofport
> in ovsdb, then ovn creates the related openflows.
>
> Fixes: 7c3f7f415f1d ("northd, controller: Flood ARP and NA packet on transit
> router.")
> Signed-off-by: Xavier Simonart <[email protected]>
> ---
Hi Xavier,
Thanks for the patch!
> tests/ovn-controller.at | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/tests/ovn-controller.at b/tests/ovn-controller.at
> index 388e5d810..1988c370b 100644
> --- a/tests/ovn-controller.at
> +++ b/tests/ovn-controller.at
> @@ -3782,6 +3782,9 @@ check ovn-sbctl chassis-add hv2 geneve 192.168.0.12 \
> check ovn-sbctl chassis-add hv3 vxlan 192.168.0.14 \
> -- set chassis hv3 other_config:is-remote=true
>
> +# Wait for tunnel to hv3 to be created.
> +OVN_WAIT_REMOTE_OUTPUT_FLOWS(["hv1"],["hv3"])
Nit: missing space after ','.
I'd also explicitly wait for flows to tunnel to hv2 here.
> +
> check ovn-nbctl --wait=hv sync
>
> chassis_cookie() {
Regards,
Dumitru
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
