[ovs-dev] [RFC PATCH v1 2/6] Vlan Support in ovn, northd changes to read logical switch network type.

[Ankur Sharma]

Hi Ben,

Sure, will merged 1,2 and 3 in a single patch.
V2 will have this change.

Regards,
Ankur

-----Original Message-----
From: Ben Pfaff <blp at ovn.org> 
Sent: Monday, February 25, 2019 9:43 AM
To: Ankur Sharma <ankur.sharma at nutanix.com>
Cc: ovs-dev at openvswitch.org
Subject: Re: [ovs-dev] [RFC PATCH v1 2/6] Vlan Support in ovn, northd changes to read logical switch network type.

On Wed, Feb 20, 2019 at 10:34:54PM +0000, Ankur Sharma wrote:
> Background:
> [1] 
> https://urldefense.proofpoint.com/v2/url?u=https-3A__mail.openvswitch.
> org_pipermail_ovs-2Ddev_2018-2DOctober_353066.html&d=DwIBAg&c=s883GpUC
> OChKOHiocYtGcg&r=mZwX9gFQgeJHzTg-68aCJgsODyUEVsHGFOfL90J6MJY&m=t2pS_PX
> 3EKDUpNOBFDics0W08_5mS4Ndo41AEObY5So&s=RhM7Na7WJLxpMpp0ZtNq9ARa37MN-sp
> 7SJEGZQ9OF_8&e= [2] 
> https://urldefense.proofpoint.com/v2/url?u=https-3A__docs.google.com_d
> ocument_d_1uoQH478wM1OZ16HrxzbOUvk5LvFnfNEWbkPT6Zmm9OU_edit-3Fusp-3Dsh
> aring&d=DwIBAg&c=s883GpUCOChKOHiocYtGcg&r=mZwX9gFQgeJHzTg-68aCJgsODyUE
> VsHGFOfL90J6MJY&m=t2pS_PX3EKDUpNOBFDics0W08_5mS4Ndo41AEObY5So&s=rU1KfK
> kURRka7dQPnH0XvgWCFviwuHXjBot8ITZSXcA&e=
> 
> This Series:
> Layer 2, Layer 3 E-W and Layer 3 N-S (NO NAT) changes for vlan backed 
> distributed logical router.

I suspect that at least patches 1 and 2, and possibly 3, should be squashed together.  Patch 1 adds a setting with no implementation, which is not what we normally do.  Patches 2 and 3 then implement the feature.