[ovs-dev] system-ovn.at: Fix a race condition.
diff mbox

Message ID 1486381208-27378-1-git-send-email-guru@ovn.org
State Accepted
Headers show

Commit Message

Gurucharan Shetty Feb. 6, 2017, 11:40 a.m. UTC
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(+)

Comments

Joe Stringer Feb. 6, 2017, 10:59 p.m. UTC | #1
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>

Patch
diff mbox

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])