[ovs-dev] [bond 3/7] bond: Add ovs-appctl bond/show to LACP sanity checks.

Ben Pfaff blp at nicira.com
Fri Jan 13 00:30:15 UTC 2012


On Thu, Jan 12, 2012 at 04:25:33PM -0800, Ethan Jackson wrote:
> > Hmm. ?The "bond - single port" test is a bit paradoxical seeing as a
> > single port isn't a bond. ?I guess it illustrates that LACP works with
> > a single port too.
> 
> Yes, I wanted to verify that a bond doesn't get created in this case.
> Would you prefer I rename it, or change the test in some way?

Do you intend to add non-LACP related bond tests?  If not, then I
think that keeping these as "lacp" tests, rather than renaming them,
is less surprising.  After all, of the two tests, both are related to
LACP, but only one is related to a bond.



More information about the dev mailing list