Commit f3103dfa00d (lacp.at: Change timing of lacp - negotiation test) fixed test failures for the lacp negotiation test on some systems, but made it crop up on others where it hadn't appeared before.
My guess is that this commit is the real fix. If it isn't, then we will probably just have to remove the test or disable it, much as I hate doing that. Signed-off-by: Ben Pfaff <b...@nicira.com> --- tests/lacp.at | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/tests/lacp.at b/tests/lacp.at index 85447c9..0bcd4b3 100644 --- a/tests/lacp.at +++ b/tests/lacp.at @@ -141,6 +141,8 @@ OVS_VSWITCHD_START( AT_CHECK([ovs-appctl netdev-dummy/set-admin-state up], 0, [OK ]) +ovs-appctl time/stop + # Wait for up to 5 (simulated) seconds, until LACP negotiation finishes. i=0 while :; do @@ -296,7 +298,7 @@ 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 25`; do ovs-appctl time/warp 100; done +for i in `seq 0 40`; do ovs-appctl time/warp 100; done AT_CHECK( [ovs-appctl lacp/show bond0 ovs-appctl lacp/show bond1 -- 1.7.2.5 _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
