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

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

 include/ovn/actions.h     |   4 +-
 lib/packets.c             |  78 ++++++++++
 lib/packets.h             |  10 ++
 ovn/lib/actions.c         |  99 +++++++++---
 ovn/northd/ovn-northd.c   | 170 ++++++++++++---------
 ovn/utilities/ovn-nbctl.c | 175 +++++++++++++++------
 tests/ovn-nbctl.at        | 379 ++++++++++++++++++++++++++++++++++++++--------
 tests/ovn.at              |   8 +-
 8 files changed, 729 insertions(+), 194 deletions(-)

Hi Mark,

On Wed,  6 Sep 2017 16:08:35 -0500
Mark Michelson <mmichels@redhat.com> 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 <jkbs@redhat.com>

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

On Thu, Sep 21, 2017 at 7:01 PM, Jakub Sitnicki <jkbs@redhat.com> wrote:

> Hi Mark,
>
> On Wed,  6 Sep 2017 16:08:35 -0500
> Mark Michelson <mmichels@redhat.com> 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 <jkbs@redhat.com>
>
> 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
> dev@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>