[ovs-dev] [PATCH] ovn-controller.at: fix occasional ovn-bridge-mappings test failure

[Han Zhou]

On Tue, May 8, 2018 at 9:02 AM, Ben Pfaff <blp at ovn.org> wrote:
>
> On Mon, May 07, 2018 at 06:17:28PM -0700, Han Zhou wrote:
> > This patch fixes the time issue in the test case so that when
external-ids
> > is updated in Open_vSwitch OVSDB, give some time for SB OVSDB to get
> > updated by ovn-controller.
> >
> > Signed-off-by: Han Zhou <hzhou8 at ebay.com>
>
> Thanks for working to make the test cases more reliable.
>
> It would be better to use OVS_WAIT_UNTIL (inside check_bridge_mappings)
> to wait until the bridge mappings getting updated, rather than using a
> fixed "sleep".  Did you consider that?
>
> Thanks,
>
> Ben.

Yes, that would be better! I updated as v2:
https://patchwork.ozlabs.org/patch/910354/

Thanks,
Han