| Message ID | 1571619079-75503-1-git-send-email-hzhou@ovn.org |
|---|---|
| Headers | show |
| Series | OVN Interconnection | expand |
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
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
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
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 >
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 >
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