[ovs-dev] [PATCH ovn 00/19] OVN Interconnection

Han Zhou hzhou at ovn.org
Mon Oct 28 17:57:40 UTC 2019


On Mon, Oct 28, 2019 at 7:25 AM Numan Siddique <numans at ovn.org> wrote:
>
> On Mon, Oct 21, 2019 at 6:23 AM Han Zhou <hzhou at ovn.org> wrote:
> >
> > The series supports interconnecting multiple OVN deployments (e.g.
 located at
> > multiple data centers) through logical routers connected with tansit
logical
> > switches with overlay tunnels, managed through OVN control plane.  See
the
> > ovn-architecture.rst document updates for more details, and find the
> > instructions in Documentation/tutorials/ovn-interconnection.rst.
> >
> > The first 6 patches are not directly related to the feature, but are
fixes for
> > existing problems but required by later patches.
> >
>
> For the first 5 patches in the series - Acked-by: Numan Siddique
> <numans at ovn.org>
>

Thanks Numan. I applied the first 5 patches to master.

> I think these 5 patches can be merged so that the v2 of this series
> will have lesser patches.
>
> Regarding the patch 6, which updates the NEWS, does it makes sense to
> delete the contents
> of this file and  start fresh capturing only OVN related news ? Or
> it's still better to preserve
> the contents ?
>

Do you mean removing all previous news, or just removing the OVS related
news and keep OVN related news?

I am ok with removing all the previous news and replace with a pointer to
OVS repo if people want to look at previous OVN news.

However, I think it may be more straightforward to leave as is for whatever
is there up to 2.12, which clearly tells how OVN is forked from the OVS
project. What do you think?

> I haven't yet looked into patch 7 onwards. I will come back to them soon.
>
> Thanks
> Numan
>
>
> > Han Zhou (19):
> >   ovn-northd.c: Fix datapath tunnel key allocation.
> >   testsuite: Use ovn-macros instead of ofproto-macros.
> >   sandbox: Remove the unused ovnnb.db from sandbox.
> >   ovn-ctl: Change the order of ovn-lib and ovs-lib.
> >   ovn-lib.in: Fix ovn-ctl status_xxx.
> >   news: Fix the out-of-date information afte split from ovs.
> >   ovn-architecture: Add documentation for OVN interconnection feature.
> >   ovn-inb: Interconnection northbound DB schema and CLI.
> >   ovn-isb: Interconnection southbound DB schema and CLI.
> >   ovn-ic: Interconnection controller with AZ registeration.
> >   ovn-northd.c: Refactor allocate_tnlid.
> >   ovn-ic: Transit switch controller.
> >   ovn-sb: Add columns is_interconn and is_remote to Chassis.
> >   ovn-ic: Interconnection gateway controller.
> >   ovn-ic: Interconnection port controller.
> >   ovn.at: e2e test for OVN interconnection.
> >   ovn-ctl: Refactor to reduce redundant code.
> >   ovn-ctl: Support commands for interconnection.
> >   tutorial: Add tutorial for OVN Interconnection.
> >
> >  .gitignore                                      |    6 +
> >  Documentation/automake.mk                       |    1 +
> >  Documentation/tutorials/index.rst               |    1 +
> >  Documentation/tutorials/ovn-interconnection.rst |  181 ++++
> >  Makefile.am                                     |    1 +
> >  NEWS                                            |   46 +-
> >  TODO.rst                                        |   10 +
> >  automake.mk                                     |   75 ++
> >  controller/binding.c                            |    6 +-
> >  controller/chassis.c                            |   14 +
> >  debian/ovn-common.install                       |    2 +
> >  debian/ovn-common.manpages                      |    4 +
> >  ic/.gitignore                                   |    2 +
> >  ic/automake.mk                                  |   10 +
> >  ic/ovn-ic.8.xml                                 |  111 +++
> >  ic/ovn-ic.c                                     | 1042
+++++++++++++++++++++++
> >  lib/.gitignore                                  |    6 +
> >  lib/automake.mk                                 |   32 +-
> >  lib/ovn-inb-idl.ann                             |    9 +
> >  lib/ovn-isb-idl.ann                             |    9 +
> >  lib/ovn-util.c                                  |   92 ++
> >  lib/ovn-util.h                                  |   15 +
> >  northd/ovn-northd.c                             |  110 +--
> >  ovn-architecture.7.xml                          |  107 ++-
> >  ovn-inb.ovsschema                               |   75 ++
> >  ovn-inb.xml                                     |  371 ++++++++
> >  ovn-isb.ovsschema                               |  129 +++
> >  ovn-isb.xml                                     |  582 +++++++++++++
> >  ovn-nb.ovsschema                                |    5 +-
> >  ovn-nb.xml                                      |   28 +-
> >  ovn-sb.ovsschema                                |    8 +-
> >  ovn-sb.xml                                      |   24 +
> >  tests/automake.mk                               |    8 +-
> >  tests/ofproto-macros.at                         |  177 ----
> >  tests/ovn-ic.at                                 |  192 +++++
> >  tests/ovn-inbctl.at                             |   65 ++
> >  tests/ovn-isbctl.at                             |  112 +++
> >  tests/ovn-macros.at                             |  161 +++-
> >  tests/ovn.at                                    |  149 ++++
> >  tests/testsuite.at                              |    4 +
> >  tutorial/ovs-sandbox                            |   81 +-
> >  utilities/.gitignore                            |    4 +
> >  utilities/automake.mk                           |   16 +
> >  utilities/ovn-ctl                               |  425 ++++++++-
> >  utilities/ovn-ctl.8.xml                         |   91 ++
> >  utilities/ovn-inbctl.8.xml                      |  174 ++++
> >  utilities/ovn-inbctl.c                          |  948
+++++++++++++++++++++
> >  utilities/ovn-isbctl.8.xml                      |  148 ++++
> >  utilities/ovn-isbctl.c                          | 1015
++++++++++++++++++++++
> >  utilities/ovn-lib.in                            |    2 +-
> >  50 files changed, 6555 insertions(+), 331 deletions(-)
> >  create mode 100644 Documentation/tutorials/ovn-interconnection.rst
> >  create mode 100644 ic/.gitignore
> >  create mode 100644 ic/automake.mk
> >  create mode 100644 ic/ovn-ic.8.xml
> >  create mode 100644 ic/ovn-ic.c
> >  create mode 100644 lib/ovn-inb-idl.ann
> >  create mode 100644 lib/ovn-isb-idl.ann
> >  create mode 100644 ovn-inb.ovsschema
> >  create mode 100644 ovn-inb.xml
> >  create mode 100644 ovn-isb.ovsschema
> >  create mode 100644 ovn-isb.xml
> >  create mode 100644 tests/ovn-ic.at
> >  create mode 100644 tests/ovn-inbctl.at
> >  create mode 100644 tests/ovn-isbctl.at
> >  create mode 100644 utilities/ovn-inbctl.8.xml
> >  create mode 100644 utilities/ovn-inbctl.c
> >  create mode 100644 utilities/ovn-isbctl.8.xml
> >  create mode 100644 utilities/ovn-isbctl.c
> >
> > --
> > 2.1.0
> >
> > _______________________________________________
> > dev mailing list
> > dev at openvswitch.org
> > https://mail.openvswitch.org/mailman/listinfo/ovs-dev


More information about the dev mailing list