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

Numan Siddique numans at ovn.org
Wed Oct 30 11:05:28 UTC 2019


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


>
> > 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