[ovs-dev] [PATCH 0/9] OVN: Support SW gateway in ovn-controller

[Ben Pfaff]

On Wed, Mar 23, 2016 at 08:01:22PM +0000, Darrell Ball wrote:
> 
> 
> 
> 
> 
> On 3/23/16, 9:00 AM, "dev on behalf of Ben Pfaff" <dev-bounces at openvswitch.org on behalf of blp at ovn.org> wrote:
> 
> >On Wed, Mar 16, 2016 at 08:15:11PM -0700, Darrell Ball wrote:
> >> The following patch series implements the SW Gateway infrastructure.
> >> The SW gateway runs in the context of ovn-controller as other HVs.
> >> The gateway node uses a single bridge (call it br-int) that is actively controlled by 
> >> OVN. This bridge also houses the tunnels connecting other HVs.
> >> Additional physical bridges are created for each logical switch supported
> >> by the gateway. These bridges act as  simple wires from the POV of >= L2
> >> on br-int and are there to support bonding at a minimum.
> >> A pair of patch ports are created to connect each physical bridge to br-int.
> >> 
> >> A new logical port type is added for SW gateways called "gw". This is needed
> >> to differentiate logic from HW gateway support. Changes to HW gateway
> >> support are coming in a next series.
> >> 
> >> patch.c: Physical bridge and patch port creation
> >> physical.c: Support gateway br-int patch ports as "physical" ports
> >> 
> >> This patch series builds (i.e. depends on) on a previous series supporting physical
> >> endpoints and their binding to logical ports. That series also added some gateway (gw)
> >> logical port checks that this series uses, in the physical.c file
> >> 
> >> binding.c support the use of physical endpoint for gateway (gw) logical ports
> >> 
> >> ovn-nb.xml; ovn-sb.xml: document the new "gw" logical port type
> >> 
> >> ovn-controller.c; patch.h: Add a chassis name parameter needed for gateways
> >> Particular logical port types are not presently specified/enforced in the NB and SB
> >> schemas themselves. This may be to allow flexibility and ease of adding new types.
> >> 
> >> ovn.at: A new test is added to exercise the SW gateway for L2 switching
> >> 
> >> Darrell Ball (8):
> >>   Add flow support for gateway patch ports in ovn-controller
> >>   Create SW gateway physical bridges and patch ports
> >>   Add chassis name parameter to support SW gateway in ovn-controller
> >>   Add chassis name parameter to support SW gateway in ovn-controller
> >>   Support physical endpoint based binding for SW gateway logical ports
> >>   Add software gateway logical port type - 'gw'
> >>   Add software gateway logical port type - 'gw'
> >>   Add a test for software gateway using ovn-controller
> >
> >This series is puzzling:
> >
> >    - The subject of this email says there are 9 patches.
> >
> >    - The summary above says that there are 8 patches.
> >
> >    - The subjects of the followups to this email say that there are 14
> >      patches, but only patches 7 through 14 are in patchwork or the
> >      archive.
> >
> >Please explain!
> >
> >Thanks,
> >
> >Ben.
> 
> Sorry for the confusion
> There was some stale state stored in Git that affected the subject and patch numbering
> that I only noticed after git send-email; I later cleaned this state and are more vigilant
> regarding Git merges :-) 
>  
> There are 8 patches in the L2 Gateway series and 6 patches in the Logical-Physical separation
> for a total of 14. The numbering of 7 to 14 is the 8 patches in the L2 gateway series that are skewed
> By 6 from the first series.
> 
> I have since sent out a single patch combining Physical-Logical separation that I’ll split into 2 patches
> soon based on subsequent feedback 

OK.

I've marked all the patches you had in patchwork as "Deferred".  I'll
wait for the next version of everything.