mbox series

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

Message ID 20171102161359.7344-1-mmichels@redhat.com
Headers show
Series Add support for IPv6 load balancers | expand

Message

Mark Michelson Nov. 2, 2017, 4:13 p.m. UTC 
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.

v2 -> v3:
* The patchset has been rebased.
* Patch 2 has been updated to get rid of a warning produced by clang 3.8
  due to some variable shadowing.

v1 -> v2:
* The patchset has been rebased and conflicts resolved. The only
  patch with noticeable differences is patch 3 of 4. This is because
  add_router_lb_flow had to be modified to not attempt to add
  undnat flows for IPv6 load balancers.
* ovn-northd manpage has been updated to detail flows that are installed
  for IPv6 load balancers. This change is in patch 3 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         |   6 +-
 lib/packets.c                 |  78 +++++++++
 lib/packets.h                 |  10 ++
 ovn/lib/actions.c             |  99 ++++++++---
 ovn/northd/ovn-northd.8.xml   |  68 +++++---
 ovn/northd/ovn-northd.c       | 182 ++++++++++++--------
 ovn/ovn-nb.xml                |  22 ++-
 ovn/utilities/ovn-nbctl.8.xml |  14 +-
 ovn/utilities/ovn-nbctl.c     | 175 ++++++++++++++-----
 tests/ovn-nbctl.at            | 379 +++++++++++++++++++++++++++++++++++-------
 tests/ovn.at                  |   8 +-
 11 files changed, 801 insertions(+), 240 deletions(-)

Comments

Ben Pfaff Nov. 2, 2017, 6:12 p.m. UTC | #1 
On Thu, Nov 02, 2017 at 11:13:55AM -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.
> 
> v2 -> v3:
> * The patchset has been rebased.
> * Patch 2 has been updated to get rid of a warning produced by clang 3.8
>   due to some variable shadowing.
> 
> v1 -> v2:
> * The patchset has been rebased and conflicts resolved. The only
>   patch with noticeable differences is patch 3 of 4. This is because
>   add_router_lb_flow had to be modified to not attempt to add
>   undnat flows for IPv6 load balancers.
> * ovn-northd manpage has been updated to detail flows that are installed
>   for IPv6 load balancers. This change is in patch 3 as well.

Thanks a lot for implementing this feature!  I applied these patches to
master.