On 2/11/11 12:37 PM, Ethan Jackson wrote:
>>
>> I doubt it. Ethan?
>>
>
> I added the hashing-algorithm along with the following logic as part
> of the lacp changes.
> I'm pretty sure it's required, but if the logic pasted below isn't
> there it won't do anything.
>
> if mode == 'lacp':
> argv += ['lacp=active']
>
> if halgo == 'src_mac':
> argv += ['bond_mode=balance-slb']
> elif halgo == "tcpudp_ports":
> argv += ['bond_mode=balance-tcp']
> else:
> log("bridge %s has invalid bond-hashing-algorithm '%s'" %
> (bridge, halgo))
> argv += ['bond_mode=balance-slb']
> elif mode in ['balance-slb', 'active-backup']:
> argv += ['lacp=off', 'bond_mode=%s' % mode]
> else:
> log("bridge %s has invalid bond-mode '%s'" % (bridge, mode))
> argv += ['lacp=off', 'bond_mode=balance-slb']
That logic is still there. I was careful not to lose your changes since
the last pull from XenServer.
-Andrew
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev_openvswitch.org
