Thanks for the reviews! I applied these to master.
On Tue, Dec 17, 2013 at 11:26:34AM -0800, Andy Zhou wrote: > Looks good. > > Acked-by: Andy Zhou <[email protected]> > > > On Fri, Dec 13, 2013 at 12:47 PM, Ben Pfaff <[email protected]> wrote: > > > These tests configured LACP and then immediately dumped out its state. > > Most of the time, this worked, but there was a brief race window in which > > the "negotiated" flag could be missing because this took one pass through > > the main loop. This fixes the problem. > > > > This race may be seen in the failures of tests 11 and 12 here: > > > > https://launchpadlibrarian.net/151884888/buildlog_ubuntu-precise-amd64.openvswitch_2.0~201309300804-1ppa1~precise_FAILEDTOBUILD.txt.gz > > > > Reported-by: Vasiliy Tolstov <[email protected]> > > Signed-off-by: Ben Pfaff <[email protected]> > > --- > > tests/lacp.at | 10 ++++++++++ > > 1 file changed, 10 insertions(+) > > > > diff --git a/tests/lacp.at b/tests/lacp.at > > index ede7c8f..d44bee0 100644 > > --- a/tests/lacp.at > > +++ b/tests/lacp.at > > @@ -6,6 +6,11 @@ OVS_VSWITCHD_START([\ > > set Port p1 lacp=active --\ > > set Interface p1 type=dummy ]) > > > > +ovs-appctl time/stop > > +ovs-appctl time/warp 100 > > +ovs-appctl time/warp 100 > > +ovs-appctl time/warp 100 > > + > > AT_CHECK([ovs-appctl lacp/show], [0], [dnl > > ---- p1 ---- > > status: active negotiated > > @@ -53,6 +58,11 @@ OVS_VSWITCHD_START([dnl > > other_config:lacp-port-priority=222 \ > > other_config:lacp-aggregation-key=3333 ]) > > > > +ovs-appctl time/stop > > +ovs-appctl time/warp 100 > > +ovs-appctl time/warp 100 > > +ovs-appctl time/warp 100 > > + > > AT_CHECK([ovs-appctl lacp/show], [0], [stdout]) > > AT_CHECK([sed -e 's/aggregation key:.*/aggregation key: <omitted>/' < > > stdout], [0], [dnl > > ---- bond ---- > > -- > > 1.7.10.4 > > > > _______________________________________________ > > dev mailing list > > [email protected] > > http://openvswitch.org/mailman/listinfo/dev > > _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
