[ovs-dev] [deprecation_v2 1/2] bond: Deprecate stable bonds.

[Ethan Jackson]

I've decided to fold the following incremental in:

diff --git a/vswitchd/vswitch.xml b/vswitchd/vswitch.xml
index f7b9f01..b71a308 100644
--- a/vswitchd/vswitch.xml
+++ b/vswitchd/vswitch.xml
@@ -833,20 +833,21 @@
       <dl>
         <dt><code>balance-tcp</code></dt>
         <dd>
           Balances flows among slaves based on L2, L3, and L4 protocol
           information such as destination MAC address, IP address, and TCP
           port.
         </dd>

         <dt><code>stable</code></dt>
         <dd>
+          <p>Deprecated and slated for removal in February 2013.</p>
           <p>Attempts to always assign a given flow to the same slave
           consistently.  In an effort to maintain stability, no load
           balancing is done.  Uses a similar hashing strategy to
           <code>balance-tcp</code>, always taking into account L3 and L4
           fields even if LACP negotiations are unsuccessful. </p>
           <p>Slave selection decisions are made based on <ref table="Interface"
           column="other_config" key="bond-stable-id"/> if set.  Otherwise,
           OpenFlow port number is used.  Decisions are consistent across all
           <code>ovs-vswitchd</code> instances with equivalent
           <ref table="Interface" column="other_config" key="bond-stable-id"/>

On Mon, Aug 20, 2012 at 11:17 AM, Ben Pfaff <blp at nicira.com> wrote:
> On Mon, Aug 20, 2012 at 11:16:05AM -0700, Ethan Jackson wrote:
>> The stable bond mode is an obsolete attempt to replicate the
>> functionality contained in the bundle action.  They are ugly and of
>> questionable usefulness.  This patch deprecates them and schedules
>> their removal for February 2013.  If there are concerns, please
>> email dev at openvswitch.org.
>>
>> Signed-off-by: Ethan Jackson <ethan at nicira.com>
>
> Looks good, thank you.