[ovs-dev,ovn,00/19] OVN Interconnection
mbox series

Message ID 1571619079-75503-1-git-send-email-hzhou@ovn.org
Headers show
Series
  • OVN Interconnection
Related show

Message

Han Zhou Oct. 21, 2019, 12:51 a.m. UTC
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.

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

Comments

Numan Siddique Oct. 28, 2019, 2:25 p.m. UTC | #1
On Mon, Oct 21, 2019 at 6:23 AM Han Zhou <hzhou@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@ovn.org>

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 ?

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@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
Han Zhou Oct. 28, 2019, 5:57 p.m. UTC | #2
On Mon, Oct 28, 2019 at 7:25 AM Numan Siddique <numans@ovn.org> wrote:
>
> On Mon, Oct 21, 2019 at 6:23 AM Han Zhou <hzhou@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@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@openvswitch.org
> > https://mail.openvswitch.org/mailman/listinfo/ovs-dev
Numan Siddique Oct. 30, 2019, 11:05 a.m. UTC | #3
On Mon, Oct 28, 2019 at 11:28 PM Han Zhou <hzhou@ovn.org> wrote:
>
> On Mon, Oct 28, 2019 at 7:25 AM Numan Siddique <numans@ovn.org> wrote:
> >
> > On Mon, Oct 21, 2019 at 6:23 AM Han Zhou <hzhou@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@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@openvswitch.org
> > > https://mail.openvswitch.org/mailman/listinfo/ovs-dev
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
Han Zhou Oct. 30, 2019, 2:43 p.m. UTC | #4
On Wed, Oct 30, 2019 at 4:05 AM Numan Siddique <numans@ovn.org> wrote:

> On Mon, Oct 28, 2019 at 11:28 PM Han Zhou <hzhou@ovn.org> wrote:
> >
> > On Mon, Oct 28, 2019 at 7:25 AM Numan Siddique <numans@ovn.org> wrote:
> > >
> > > On Mon, Oct 21, 2019 at 6:23 AM Han Zhou <hzhou@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@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@openvswitch.org
> > > > https://mail.openvswitch.org/mailman/listinfo/ovs-dev
> > _______________________________________________
> > dev mailing list
> > dev@openvswitch.org
> > https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
Numan Siddique Oct. 30, 2019, 2:50 p.m. UTC | #5
On Wed, Oct 30, 2019, 8:14 PM Han Zhou <hzhou@ovn.org> wrote:

> On Wed, Oct 30, 2019 at 4:05 AM Numan Siddique <numans@ovn.org> wrote:
>
> > On Mon, Oct 28, 2019 at 11:28 PM Han Zhou <hzhou@ovn.org> wrote:
> > >
> > > On Mon, Oct 28, 2019 at 7:25 AM Numan Siddique <numans@ovn.org> wrote:
> > > >
> > > > On Mon, Oct 21, 2019 at 6:23 AM Han Zhou <hzhou@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@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.
>

Sure.
Acked-by: Numan Siddique <numans@ovn.org>
for patch 6

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@openvswitch.org
> > > > > https://mail.openvswitch.org/mailman/listinfo/ovs-dev
> > > _______________________________________________
> > > dev mailing list
> > > dev@openvswitch.org
> > > https://mail.openvswitch.org/mailman/listinfo/ovs-dev
> >
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
Han Zhou Oct. 30, 2019, 9:15 p.m. UTC | #6
On Wed, Oct 30, 2019 at 7:51 AM Numan Siddique <numans@ovn.org> wrote:
>
>
>
> On Wed, Oct 30, 2019, 8:14 PM Han Zhou <hzhou@ovn.org> wrote:
>>
>> On Wed, Oct 30, 2019 at 4:05 AM Numan Siddique <numans@ovn.org> wrote:
>>
>> > On Mon, Oct 28, 2019 at 11:28 PM Han Zhou <hzhou@ovn.org> wrote:
>> > >
>> > > On Mon, Oct 28, 2019 at 7:25 AM Numan Siddique <numans@ovn.org>
wrote:
>> > > >
>> > > > On Mon, Oct 21, 2019 at 6:23 AM Han Zhou <hzhou@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@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.
>
>
> Sure.
> Acked-by: Numan Siddique <numans@ovn.org>
> for patch 6
>
> Numan
>
Thanks. I applied patch 6 to master after rebase.
I sent v2 for the rest of the series at:
https://patchwork.ozlabs.org/project/openvswitch/list/?series=139731

Thanks,
Han