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

Han Zhou hzhou at ovn.org
Wed Oct 30 14:43:15 UTC 2019


On Wed, Oct 30, 2019 at 4:05 AM Numan Siddique <numans at ovn.org> wrote:

> On Mon, Oct 28, 2019 at 11:28 PM Han Zhou <hzhou at ovn.org> wrote:
> >
> > 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?
>
> Ok. I think this makes more sense. Let's keep it AS IS.
>
> Thanks
> Numan
>

Thanks Numan. So can I consider this as acked for patch 6, so that I can
merge it and send v2 for the rest of the patches.

>
>
> >
> > > 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
> > _______________________________________________
> > dev mailing list
> > dev at openvswitch.org
> > https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>


More information about the dev mailing list