[ovs-dev] [PATCH] vswitch.xml: Fix bond-hash-basis documentation placement.
blp at nicira.com
Wed Nov 30 19:46:44 PST 2011
Great, add a period to the final sentence and ship it. Thanks.
On Wed, Nov 30, 2011 at 07:41:34PM -0800, Ethan Jackson wrote:
> Alright, I've changed the wording to:
> + An integer hashed along with flows when choosing output slaves in load
> + balanced bonds. When changed, all flows will be assigned different
> + hash values possibly causing slave selection decisions to change. Does
> + not affect bonding modes which do not employ load balancing such as
> + <code>active-backup</code>
> On Wed, Nov 30, 2011 at 19:21, Ben Pfaff <blp at nicira.com> wrote:
> > On Wed, Nov 30, 2011 at 06:09:39PM -0800, Ethan Jackson wrote:
> >> The bond-hash-basis other_config column was documented under "LACP
> >> Configuration". ?Instead it should be documented as general bonding
> >> configuration.
> > Looks good but it might be worth adding a note that it does not affect
> > the active-backup mode (right?).
> > Thanks,
> > Ben.
More information about the dev