[ovs-dev] [ovn-controller-vtep V4 5/6] ovn-controller-vtep: Add binding module.

[Russell Bryant]

On 07/22/2015 01:13 PM, Alex Wang wrote:
> Thx a lot for the review,
> 
> 
> On Tue, Jul 21, 2015 at 1:35 PM, Russell Bryant <rbryant at redhat.com
> <mailto:rbryant at redhat.com>> wrote:
> 
>     On 07/16/2015 03:56 AM, Alex Wang wrote:
>     > This commit adds the binding module to ovn-controller-vtep.  The
>     > module will scan through the Binding table in ovnsb.  If there is
>     > a binding for a logical port in the vtep gateway chassis's
>     > "vtep_logical_switches" map, sets the binding's chassis column to the
>     > vtep gateway chassis.
>     >
>     > Signed-off-by: Alex Wang <alexw at nicira.com <mailto:alexw at nicira.com>>
> 
>     As discussed before, I'd like to see this done using logical port type
>     and options instead of a special name, but this could be reworked later
>     if it merges before the addition of type and options.
> 
>     A few comments inline ..
> 
> 
> Yeah, I think this patch and my last one could just sit in my repo for a bit
> longer...  Once your change is committed, I'll rebase and resubmit~

To be clear, I have no problem with this merging first.  It was ready
first and there aren't actually any conflicts.  It'll just introduce an
alternative way to implement part of this and we can convert it later if
needed.

-- 
Russell Bryant