Message ID | 1486381208-27378-1-git-send-email-guru@ovn.org |
---|---|
State | Accepted |
Headers | show |
On 6 February 2017 at 03:40, Gurucharan Shetty <guru@ovn.org> wrote: > Commit 2f35b6c5c12c (system-ovn.at: Fix race conditions.) > fixed a number of race conditions, but left atleast one out. > This commit fixes that race condition. > > Reported-by: Joe Stringer <joe@ovn.org> > Signed-off-by: Gurucharan Shetty <guru@ovn.org> Thanks! Acked-by: Joe Stringer <joe@ovn.org>
diff --git a/tests/system-ovn.at b/tests/system-ovn.at index 7296550..dd62bd1 100644 --- a/tests/system-ovn.at +++ b/tests/system-ovn.at @@ -134,6 +134,9 @@ icmp,orig=(src=192.168.2.2,dst=172.16.1.2,id=<cleared>,type=8,code=0),reply=(src # Add static routes to handle east-west NAT. ovn-nbctl lr-route-add R1 30.0.0.0/24 20.0.0.2 +# wait for ovn-controller to catch up. +ovn-nbctl --wait=hv sync + # Flush conntrack entries for easier output parsing of next test. AT_CHECK([ovs-appctl dpctl/flush-conntrack])
Commit 2f35b6c5c12c (system-ovn.at: Fix race conditions.) fixed a number of race conditions, but left atleast one out. This commit fixes that race condition. Reported-by: Joe Stringer <joe@ovn.org> Signed-off-by: Gurucharan Shetty <guru@ovn.org> --- tests/system-ovn.at | 3 +++ 1 file changed, 3 insertions(+)