On Wed, Oct 30, 2019, 8:14 PM Han Zhou <hz...@ovn.org> wrote: > On Wed, Oct 30, 2019 at 4:05 AM Numan Siddique <num...@ovn.org> wrote: > > > On Mon, Oct 28, 2019 at 11:28 PM Han Zhou <hz...@ovn.org> wrote: > > > > > > On Mon, Oct 28, 2019 at 7:25 AM Numan Siddique <num...@ovn.org> wrote: > > > > > > > > On Mon, Oct 21, 2019 at 6:23 AM Han Zhou <hz...@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 > > > > <num...@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 <num...@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 > > > > > d...@openvswitch.org > > > > > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > > > _______________________________________________ > > > dev mailing list > > > d...@openvswitch.org > > > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > > > _______________________________________________ > dev mailing list > d...@openvswitch.org > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev