On Thu, Sep 17, 2026 at 1:16 PM Dumitru Ceara via dev <
[email protected]> wrote:
> The test incorrectly assumed that "ovn-ic-nbctl --wait=sb <command>"
> waits for the local Southbound databases to be up to date. That's not
> the case, as documented it:
>
> With <code>--wait=sb</code>, before <code>ovn-ic-nbctl</code> exits,
> it waits for <code>ovn-ics</code> to bring the Interconnect
> southbound database up-to-date with the Interconnect northbound
> database updates.
>
> Fixes: e5565a990ee5 ("ic: Add Transit Router port support.")
> Signed-off-by: Dumitru Ceara <[email protected]>
> ---
> tests/ovn-ic.at | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tests/ovn-ic.at b/tests/ovn-ic.at
> index 51ac645b47..80db1a1c28 100644
> --- a/tests/ovn-ic.at
> +++ b/tests/ovn-ic.at
> @@ -5749,10 +5749,10 @@ check_row_count nb:Logical_Router_Port 1 networks="
> 192.168.10.10/24 192.168.10.2
>
> ovn_as az1
> check ovn-ic-nbctl --wait=sb tr-del tr0
> -check_row_count sb:Datapath_Binding 0
> +wait_row_count Datapath_Binding 0
>
> ovn_as az2
> -check_row_count sb:Datapath_Binding 0
> +wait_row_count Datapath_Binding 0
>
> OVN_CLEANUP_IC([az1], [az2])
> AT_CLEANUP
> --
> 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
