mbox series

[ovs-dev,ovn,v2,0/6] Add OVN IPv6 support

Message ID 20191030145324.9988-1-russell@ovn.org
Headers show
Series Add OVN IPv6 support | expand

Message

Russell Bryant Oct. 30, 2019, 2:53 p.m. UTC 
v1 -> v2:
 - Address checkpatch warnings about line length
 - Fix a sparse error in ovn-nbctl
 - Add lflow docs in ovn-northd.8.xml

[PATCH ovn v2 1/6] northd: Fix table ID for IPv6 router ingress.
[PATCH ovn v2 2/6] actions: Add IPv6 support to lflow NAT actions
[PATCH ovn v2 3/6] ovn-nbctl: Allow IPv6 NAT rules to be added
[PATCH ovn v2 4/6] northd: Add lflows for IPv6 NAT.
[PATCH ovn v2 5/6] system-ovn: Add IPv6 NAT test cases
[PATCH ovn v2 6/6] NEWS: Add IPv6 NAT support

 NEWS                    |    6 
 include/ovn/actions.h   |    6 
 lib/actions.c           |   35 +
 northd/ovn-northd.8.xml |  233 +++++++-----
 northd/ovn-northd.c     |  386 +++++++++++++++------
 tests/ovn-nbctl.at      |   41 +-
 tests/ovn.at            |   18 -
 tests/system-ovn.at     |  862 +++++++++++++++++++++++++++++++++++++++++++++++-
 utilities/ovn-nbctl.c   |   49 ++
 utilities/ovn-trace.c   |   15 
 10 files changed, 1408 insertions(+), 243 deletions(-)

--
Russell Bryant

Comments

Numan Siddique Oct. 30, 2019, 3:56 p.m. UTC | #1 
On Wed, Oct 30, 2019 at 8:25 PM Russell Bryant <russell@ovn.org> wrote:
>
> v1 -> v2:
>  - Address checkpatch warnings about line length
>  - Fix a sparse error in ovn-nbctl
>  - Add lflow docs in ovn-northd.8.xml
>
> [PATCH ovn v2 1/6] northd: Fix table ID for IPv6 router ingress.
> [PATCH ovn v2 2/6] actions: Add IPv6 support to lflow NAT actions
> [PATCH ovn v2 3/6] ovn-nbctl: Allow IPv6 NAT rules to be added
> [PATCH ovn v2 4/6] northd: Add lflows for IPv6 NAT.
> [PATCH ovn v2 5/6] system-ovn: Add IPv6 NAT test cases
> [PATCH ovn v2 6/6] NEWS: Add IPv6 NAT support

Thanks for addressing the comments in v2.

Acked-by: Numan Siddique <numans@ovn.org> for the whole series.

Numan

>
>  NEWS                    |    6
>  include/ovn/actions.h   |    6
>  lib/actions.c           |   35 +
>  northd/ovn-northd.8.xml |  233 +++++++-----
>  northd/ovn-northd.c     |  386 +++++++++++++++------
>  tests/ovn-nbctl.at      |   41 +-
>  tests/ovn.at            |   18 -
>  tests/system-ovn.at     |  862 +++++++++++++++++++++++++++++++++++++++++++++++-
>  utilities/ovn-nbctl.c   |   49 ++
>  utilities/ovn-trace.c   |   15
>  10 files changed, 1408 insertions(+), 243 deletions(-)
>
> --
> Russell Bryant
>
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
Russell Bryant Oct. 30, 2019, 4:31 p.m. UTC | #2 
On Wed, Oct 30, 2019 at 11:56 AM Numan Siddique <numans@ovn.org> wrote:

> On Wed, Oct 30, 2019 at 8:25 PM Russell Bryant <russell@ovn.org> wrote:
> >
> > v1 -> v2:
> >  - Address checkpatch warnings about line length
> >  - Fix a sparse error in ovn-nbctl
> >  - Add lflow docs in ovn-northd.8.xml
> >
> > [PATCH ovn v2 1/6] northd: Fix table ID for IPv6 router ingress.
> > [PATCH ovn v2 2/6] actions: Add IPv6 support to lflow NAT actions
> > [PATCH ovn v2 3/6] ovn-nbctl: Allow IPv6 NAT rules to be added
> > [PATCH ovn v2 4/6] northd: Add lflows for IPv6 NAT.
> > [PATCH ovn v2 5/6] system-ovn: Add IPv6 NAT test cases
> > [PATCH ovn v2 6/6] NEWS: Add IPv6 NAT support
>
> Thanks for addressing the comments in v2.
>
> Acked-by: Numan Siddique <numans@ovn.org> for the whole series.
>

Thanks for the fast reviews!  I pushed this series to master.

>
> Numan
>
> >
> >  NEWS                    |    6
> >  include/ovn/actions.h   |    6
> >  lib/actions.c           |   35 +
> >  northd/ovn-northd.8.xml |  233 +++++++-----
> >  northd/ovn-northd.c     |  386 +++++++++++++++------
> >  tests/ovn-nbctl.at      |   41 +-
> >  tests/ovn.at            |   18 -
> >  tests/system-ovn.at     |  862
> +++++++++++++++++++++++++++++++++++++++++++++++-
> >  utilities/ovn-nbctl.c   |   49 ++
> >  utilities/ovn-trace.c   |   15
> >  10 files changed, 1408 insertions(+), 243 deletions(-)
> >
> > --
> > Russell Bryant
> >
> > _______________________________________________
> > dev mailing list
> > dev@openvswitch.org
> > https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>