Re: [ovs-dev] [PATCH 0/4] ovn: Add support for IPv6 load balancers

2017-09-21 Thread Numan Siddique
On Thu, Sep 21, 2017 at 7:01 PM, Jakub Sitnicki  wrote:

> Hi Mark,
>
> On Wed,  6 Sep 2017 16:08:35 -0500
> Mark Michelson  wrote:
>
> > This patchset adds the necessary items in order to support IPv6 load
> > balancers in OVN. No syntax has changed in ovn-nbctl or in the
> > northbound database to support this. Appropriate tests have been
> > added to the testsuite as well.
> >
> > Mark Michelson (4):
> >   Add general-purpose IP/port parsing function.
> >   ovn: Allow ct_lb actions to take IPv6 address arguments.
> >   ovn: Allow northd to install IPv6 ct_lb logical flows.
> >   ovn: Add IPv6 capability to ovn-nbctl lb-add
>
> Tested it in a setup as described in:
>
>   http://blog.spinhirne.com/2016/09/the-ovn-load-balancer.html
>
> but with overlay and external networks using IPv6.
>
> I've tried creating an LB both with and w/o a port numbers. All worked
> fine. For example:
>
> ovn-nbctl create load_balancer 'vips:"fd66::254"="fc11::130,fc11::131"'
> ovn-nbctl create load_balancer 'vips:"[fc11::62]:80"="[fc11::
> 130]:8000,[fc11::131]:8000"'
>
> Feel free to add my:
>
> Tested-by: Jakub Sitnicki 
>
> However, ovn-nb man-page section for Load_Balancer table needs an update
> because right now it says that IPv4 addresses are expected in vips
> mapping.
>

As Jakub mentioned, this patch series requires necessary changes to
ovn/northd/ovn-northd.8.xml with the appropriate documentation.

Thanks
Numan



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


Re: [ovs-dev] [PATCH 0/4] ovn: Add support for IPv6 load balancers

2017-09-21 Thread Jakub Sitnicki
Hi Mark,

On Wed,  6 Sep 2017 16:08:35 -0500
Mark Michelson  wrote:

> This patchset adds the necessary items in order to support IPv6 load
> balancers in OVN. No syntax has changed in ovn-nbctl or in the
> northbound database to support this. Appropriate tests have been
> added to the testsuite as well.
> 
> Mark Michelson (4):
>   Add general-purpose IP/port parsing function.
>   ovn: Allow ct_lb actions to take IPv6 address arguments.
>   ovn: Allow northd to install IPv6 ct_lb logical flows.
>   ovn: Add IPv6 capability to ovn-nbctl lb-add

Tested it in a setup as described in:

  http://blog.spinhirne.com/2016/09/the-ovn-load-balancer.html

but with overlay and external networks using IPv6.

I've tried creating an LB both with and w/o a port numbers. All worked
fine. For example:

ovn-nbctl create load_balancer 'vips:"fd66::254"="fc11::130,fc11::131"'
ovn-nbctl create load_balancer 
'vips:"[fc11::62]:80"="[fc11::130]:8000,[fc11::131]:8000"'

Feel free to add my:

Tested-by: Jakub Sitnicki 

However, ovn-nb man-page section for Load_Balancer table needs an update
because right now it says that IPv4 addresses are expected in vips
mapping.

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