On Thu, Mar 28, 2013 at 01:44:36PM -0700, Ansis Atteka wrote: > On Thu, Mar 28, 2013 at 11:47 AM, Ben Pfaff <b...@nicira.com> wrote: > > 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 > > > > Looks good to me. At least fixed test issue in my setup.
Thank you. I applied this to master. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
