On Tue, Jul 1, 2025 at 2:20 PM Dumitru Ceara via dev <
[email protected]> wrote:

> The test assumed the first transaction to be logged by ovn-northd is
> the one corresponding to the switch datapath creation.  In some cases
> however [0], likely on slower systems, the first transaction through
> which ovn-northd synchronizes the initial state (e.g., RBAC) was also
> happening after jsonrpc logging was enabled by the test.
>
> Wait for a full SB sync before running the actual test.
>
> [0]
> https://github.com/dceara/ovn/actions/runs/15995705960/job/45118770130#step:12:5400
>
> Fixes: e1495fbc86bb ("northd: Remove delay in sb_cfg propagation.")
> Signed-off-by: Dumitru Ceara <[email protected]>
> ---
>  tests/ovn-northd.at | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/tests/ovn-northd.at b/tests/ovn-northd.at
> index 5e0747fcc8..8204a5cc35 100644
> --- a/tests/ovn-northd.at
> +++ b/tests/ovn-northd.at
> @@ -17436,6 +17436,9 @@ OVN_FOR_EACH_NORTHD_NO_HV([
>  AT_SETUP([sb_cfg propagation])
>  ovn_start
>
> +# Make sure ovn-northd propagated all initial state to SB (e.g., RBAC).
> +check ovn-nbctl --wait=sb sync
> +
>  #
>  # Test engine call does not happen between sb db transaction
>  # commitment and sb_cfg write to the nb db (if have to)
> --
> 2.49.0
>
> _______________________________________________
> dev mailing list
> [email protected]
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
>
Thank you Dumitru,

I went ahead and merged this into main.

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

Reply via email to