From f3103dfa00d8c7c30af86cd861cca538caef4c15 Mon Sep 17 00:00:00 2001 From: Jarno Rajahalme Date: Thu, 28 Mar 2013 15:47:46 +0200 Subject: [PATCH] lacp.at: Change timing of lacp - negotiation test 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 Signed-off-by: Ben Pfaff --- tests/lacp.at | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/lacp.at b/tests/lacp.at index 49f3bb1a3..85447c9aa 100644 --- a/tests/lacp.at +++ b/tests/lacp.at @@ -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 -- 2.43.0