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 <b...@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.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to