lacp.at: Change timing of lacp - negotiation test
authorJarno Rajahalme <jarno.rajahalme@nsn.com>
Thu, 28 Mar 2013 13:47:46 +0000 (15:47 +0200)
committerBen Pfaff <blp@nicira.com>
Thu, 28 Mar 2013 14:55:32 +0000 (07:55 -0700)
The "lacp - negotiation" test (# 3) used to always fail when running
multiple tests in parallel (-j2).  The reduced simulated wait time
(from 4 seconds to 2.5 seconds) seems to still give the expected result,
and works also when running multiple tests in parallel.

Signed-off-by: Jarno Rajahalme <jarno.rajahalme@nsn.com>
Signed-off-by: Ben Pfaff <blp@nicira.com>
tests/lacp.at

index 49f3bb1..85447c9 100644 (file)
@@ -290,13 +290,13 @@ AT_CHECK([grep 'active slave' stdout], [0], [dnl
 ])
 
 # Redirect the patch link between p0 and p2 so that no packets get
-# back and forth across them anymore.  Then wait 4 simulated
+# back and forth across them anymore.  Then wait 2.5 simulated
 # seconds.  The LACP state should become "expired" for p0 and p2.
 AT_CHECK([ovs-vsctl \
 -- add-port br0 null0 -- set int null0 type=patch options:peer=p2 -- set int p2 options:peer=null0 \
 -- add-port br1 null1 -- set int null1 type=patch options:peer=p0 -- set int p0 options:peer=null1])
 
-for i in `seq 0 40`; do ovs-appctl time/warp 100; done
+for i in `seq 0 25`; do ovs-appctl time/warp 100; done
 AT_CHECK(
   [ovs-appctl lacp/show bond0
 ovs-appctl lacp/show bond1