tests: Add delay in NetFlow unit tests before killing processes.
authorJesse Gross <jesse@nicira.com>
Tue, 7 Aug 2012 22:26:33 +0000 (15:26 -0700)
committerJesse Gross <jesse@nicira.com>
Tue, 14 Aug 2012 00:16:57 +0000 (17:16 -0700)
At the end of the NetFlow unit tests we warp time to force any
remaining flows to expire and then immediately kill OVS and the
collector.  However, this creates a race where sometimes these
processes are killed before the last records are sent or collected.
It's possible to force OVS to go through the run loop one last time
before exiting but it's harder to enforce that the collector receives
the packet.  This simply avoids the problem by adding a 1 second delay
before killing the processes, which should be more than enough time.

Signed-off-by: Jesse Gross <jesse@nicira.com>
tests/ofproto-dpif.at

index 300d091a041eaf4abd9371085d02f958b3c23b72..881831648e782a623d47b5ba9301802bcc02007d 100644 (file)
@@ -958,6 +958,7 @@ for delay in 1000 30000; do
     ovs-appctl time/warp $delay
 done
 
+sleep 1
 OVS_VSWITCHD_STOP
 ovs-appctl -t test-netflow exit
 
@@ -1006,6 +1007,7 @@ done
 
 ovs-appctl time/warp 10000
 
+sleep 1
 OVS_VSWITCHD_STOP
 ovs-appctl -t test-netflow exit