Re: [ovs-dev] [PATCH v2 1/4] ovn: l3ha, handling of multiple gateways

2017-06-08 Thread Ben Pfaff
On Thu, Jun 08, 2017 at 02:05:05PM +, majop...@redhat.com wrote:
> From: Miguel Angel Ajo 
> 
> This patch handles multiple gateways with priorities in chassisredirect
> ports, any gateway with a chassis redirect port will implement the
> rules to de-encapsulate incomming packets for such port.
> 
> And hosts targetting a remote chassisredirect port will setup a
> bundle(active_backup, ..) action to each tunnel port, in the given
> priority order.
> 
> Signed-off-by: Miguel Angel Ajo 

I feel unqualified to fully and properly review this series.  Guru, is
it something you'd feel able to take a look at?  Is anyone else planning
to review this?

Thanks,

Ben.
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


Re: [ovs-dev] [PATCH v2 1/4] ovn: l3ha, handling of multiple gateways

2017-06-09 Thread Miguel Angel Ajo Pelayo
Thank you Ben and Guru,

I plan to generalise this work with follow up patches, to let us
setup a multiple chassis on a Logical_Router too (and hence handle it
via the l3gateway ports).

   Anil Venkata is also working on a follow up patch to make sure
gARPs are only  announced on the master/active gateway.


On Thu, Jun 8, 2017 at 11:39 PM, Ben Pfaff  wrote:

> On Thu, Jun 08, 2017 at 02:05:05PM +, majop...@redhat.com wrote:
> > From: Miguel Angel Ajo 
> >
> > This patch handles multiple gateways with priorities in chassisredirect
> > ports, any gateway with a chassis redirect port will implement the
> > rules to de-encapsulate incomming packets for such port.
> >
> > And hosts targetting a remote chassisredirect port will setup a
> > bundle(active_backup, ..) action to each tunnel port, in the given
> > priority order.
> >
> > Signed-off-by: Miguel Angel Ajo 
>
> I feel unqualified to fully and properly review this series.  Guru, is
> it something you'd feel able to take a look at?  Is anyone else planning
> to review this?
>
> Thanks,
>
> Ben.
>
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


Re: [ovs-dev] [PATCH v2 1/4] ovn: l3ha, handling of multiple gateways

2017-06-09 Thread Miguel Angel Ajo Pelayo
I'd like this feature to be considered for OVS 2.8,

I know the deadline for branching is close, but this would let us implement
l3ha on the networking-ovn side, closing one of the gaps we have
with the reference implementation now.


Best regards,
Miguel Ángel

On Fri, Jun 9, 2017 at 11:35 AM, Miguel Angel Ajo Pelayo <
majop...@redhat.com> wrote:

> Thank you Ben and Guru,
>
> I plan to generalise this work with follow up patches, to let us
> setup a multiple chassis on a Logical_Router too (and hence handle it
> via the l3gateway ports).
>
>Anil Venkata is also working on a follow up patch to make sure
> gARPs are only  announced on the master/active gateway.
>
>
> On Thu, Jun 8, 2017 at 11:39 PM, Ben Pfaff  wrote:
>
>> On Thu, Jun 08, 2017 at 02:05:05PM +, majop...@redhat.com wrote:
>> > From: Miguel Angel Ajo 
>> >
>> > This patch handles multiple gateways with priorities in chassisredirect
>> > ports, any gateway with a chassis redirect port will implement the
>> > rules to de-encapsulate incomming packets for such port.
>> >
>> > And hosts targetting a remote chassisredirect port will setup a
>> > bundle(active_backup, ..) action to each tunnel port, in the given
>> > priority order.
>> >
>> > Signed-off-by: Miguel Angel Ajo 
>>
>> I feel unqualified to fully and properly review this series.  Guru, is
>> it something you'd feel able to take a look at?  Is anyone else planning
>> to review this?
>>
>> Thanks,
>>
>> Ben.
>>
>
>
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


Re: [ovs-dev] [PATCH v2 1/4] ovn: l3ha, handling of multiple gateways

2017-06-09 Thread Guru Shetty
On 8 June 2017 at 14:39, Ben Pfaff  wrote:

> On Thu, Jun 08, 2017 at 02:05:05PM +, majop...@redhat.com wrote:
> > From: Miguel Angel Ajo 
> >
> > This patch handles multiple gateways with priorities in chassisredirect
> > ports, any gateway with a chassis redirect port will implement the
> > rules to de-encapsulate incomming packets for such port.
> >
> > And hosts targetting a remote chassisredirect port will setup a
> > bundle(active_backup, ..) action to each tunnel port, in the given
> > priority order.
> >
> > Signed-off-by: Miguel Angel Ajo 
>
> I feel unqualified to fully and properly review this series.  Guru, is
> it something you'd feel able to take a look at?  Is anyone else planning
> to review this?
>

I will have a go at it.

>
> Thanks,
>
> Ben.
>
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


Re: [ovs-dev] [PATCH v2 1/4] ovn: l3ha, handling of multiple gateways

2017-06-13 Thread Russell Bryant
On Fri, Jun 9, 2017 at 10:46 AM, Guru Shetty  wrote:
> On 8 June 2017 at 14:39, Ben Pfaff  wrote:
>
>> On Thu, Jun 08, 2017 at 02:05:05PM +, majop...@redhat.com wrote:
>> > From: Miguel Angel Ajo 
>> >
>> > This patch handles multiple gateways with priorities in chassisredirect
>> > ports, any gateway with a chassis redirect port will implement the
>> > rules to de-encapsulate incomming packets for such port.
>> >
>> > And hosts targetting a remote chassisredirect port will setup a
>> > bundle(active_backup, ..) action to each tunnel port, in the given
>> > priority order.
>> >
>> > Signed-off-by: Miguel Angel Ajo 
>>
>> I feel unqualified to fully and properly review this series.  Guru, is
>> it something you'd feel able to take a look at?  Is anyone else planning
>> to review this?
>>
>
> I will have a go at it.

Thanks a lot, Guru.  Since this is important for OpenStack, let me
know if you won't have time to review and I'll make time to help.  I
think you're a better reviewer for this one, though.

-- 
Russell Bryant
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


Re: [ovs-dev] [PATCH v2 1/4] ovn: l3ha, handling of multiple gateways

2017-06-13 Thread Guru Shetty
On 13 June 2017 at 10:17, Russell Bryant  wrote:

> On Fri, Jun 9, 2017 at 10:46 AM, Guru Shetty  wrote:
> > On 8 June 2017 at 14:39, Ben Pfaff  wrote:
> >
> >> On Thu, Jun 08, 2017 at 02:05:05PM +, majop...@redhat.com wrote:
> >> > From: Miguel Angel Ajo 
> >> >
> >> > This patch handles multiple gateways with priorities in
> chassisredirect
> >> > ports, any gateway with a chassis redirect port will implement the
> >> > rules to de-encapsulate incomming packets for such port.
> >> >
> >> > And hosts targetting a remote chassisredirect port will setup a
> >> > bundle(active_backup, ..) action to each tunnel port, in the given
> >> > priority order.
> >> >
> >> > Signed-off-by: Miguel Angel Ajo 
> >>
> >> I feel unqualified to fully and properly review this series.  Guru, is
> >> it something you'd feel able to take a look at?  Is anyone else planning
> >> to review this?
> >>
> >
> > I will have a go at it.
>
> Thanks a lot, Guru.  Since this is important for OpenStack, let me
> know if you won't have time to review and I'll make time to help.  I
> think you're a better reviewer for this one, though.
>
Russell,
 If you want to do the initial round, please go ahead. Since you know more
about the usecase and how it is to be used by OpenStack, your review will
help.


>
> --
> Russell Bryant
>
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


Re: [ovs-dev] [PATCH v2 1/4] ovn: l3ha, handling of multiple gateways

2017-06-13 Thread Russell Bryant
On Tue, Jun 13, 2017 at 1:22 PM, Guru Shetty  wrote:
>
>
> On 13 June 2017 at 10:17, Russell Bryant  wrote:
>>
>> On Fri, Jun 9, 2017 at 10:46 AM, Guru Shetty  wrote:
>> > On 8 June 2017 at 14:39, Ben Pfaff  wrote:
>> >
>> >> On Thu, Jun 08, 2017 at 02:05:05PM +, majop...@redhat.com wrote:
>> >> > From: Miguel Angel Ajo 
>> >> >
>> >> > This patch handles multiple gateways with priorities in
>> >> > chassisredirect
>> >> > ports, any gateway with a chassis redirect port will implement the
>> >> > rules to de-encapsulate incomming packets for such port.
>> >> >
>> >> > And hosts targetting a remote chassisredirect port will setup a
>> >> > bundle(active_backup, ..) action to each tunnel port, in the given
>> >> > priority order.
>> >> >
>> >> > Signed-off-by: Miguel Angel Ajo 
>> >>
>> >> I feel unqualified to fully and properly review this series.  Guru, is
>> >> it something you'd feel able to take a look at?  Is anyone else
>> >> planning
>> >> to review this?
>> >>
>> >
>> > I will have a go at it.
>>
>> Thanks a lot, Guru.  Since this is important for OpenStack, let me
>> know if you won't have time to review and I'll make time to help.  I
>> think you're a better reviewer for this one, though.
>
> Russell,
>  If you want to do the initial round, please go ahead. Since you know more
> about the usecase and how it is to be used by OpenStack, your review will
> help.

OK - I'll do a pass on them.

-- 
Russell Bryant
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev