On Wed, Mar 27, 2019 at 12:06 PM Han Zhou <zhou...@gmail.com> wrote: > > On Wed, Mar 27, 2019 at 11:34 AM <nusid...@redhat.com> wrote: > > > > From: Numan Siddique <nusid...@redhat.com> > > > > This patch series adds a generic HA chassis group support in OVN > > deprecating the existing Gateway chassis support. The final patch > > of the series adds the 'external' port support in OVN. > > The 'external' port patch addresses the review comments from Han Zhou > > which he provided when 'external' port patch was submitted without > > the HA support. > > > > A generic HA chassis group support is added so that both the distributed > > logical router ports (providing gateway functionality) and 'external' > > ports can use it for HA and also to simplify the existing HA code > > (which seems to be a bit complicated). > > > > To support HA, BFD is configured on tunnel ports. And even though > > 'external' ports are expected to be used with the logical > > switches having localnet ports (representing physical networks), > > BFD is used for now since each chassis uses geneve tunnels with > > all other chassis in the OVN cluster. > > > > > > v5 -> v6 > > ---- > > * Addressed review comments > > - Changed the HA_Chassis_Group reference in Logical_Switch_Port to > > strong. > > > > v4 -> v5 > > ----- > > * Addressed review commemts from Han > > * Resolved merge conflicts > > > > v3 -> v4 > > ----- > > * Addressed the review comments from Han on patch 2. > > > > v2 -> v3 > > ------ > > * Addressed the review comments from Han on patch 2. > > - Added the code to set the ref_chassis for cases where > > a logical port can reach a gateway router port via indirect > > connections. > > - Removed the deprecation message about Gateway_chassis in > > ovn-nb.xml > > > > * Rebased few patches to resolve merge conflicts. > > > > v1 -> v2 > > ------- > > * Rebased as there were merge conflicts > > * Addressed the review comment from Han in the patch 1. > > > > Numan Siddique (5): > > ovn-northd: Reuse the hmaps - datapaths and ports in ovnsb_db_run() > > ovn: Add generic HA chassis group > > ovn-controller: Make use of ha_chassis_group table to bind the > > chassisredirect ports > > ovn-northd: Delete the references to gateway_chasss in SB DB > > ovn: Support a new Logical_Switch_Port.type - 'external' > > > > NEWS | 2 + > > ovn/controller/automake.mk | 4 +- > > ovn/controller/bfd.c | 229 +++---- > > ovn/controller/bfd.h | 15 +- > > ovn/controller/binding.c | 31 +- > > ovn/controller/binding.h | 1 - > > ovn/controller/gchassis.c | 222 ------- > > ovn/controller/gchassis.h | 71 --- > > ovn/controller/ha-chassis.c | 203 ++++++ > > ovn/controller/ha-chassis.h | 50 ++ > > ovn/controller/lflow.c | 29 +- > > ovn/controller/lflow.h | 3 +- > > ovn/controller/ovn-controller.c | 14 +- > > ovn/controller/physical.c | 109 ++-- > > ovn/controller/physical.h | 3 +- > > ovn/controller/pinctrl.c | 38 +- > > ovn/controller/pinctrl.h | 1 - > > ovn/lib/chassis-index.c | 26 + > > ovn/lib/chassis-index.h | 4 + > > ovn/lib/ovn-util.c | 1 + > > ovn/northd/ovn-northd.8.xml | 37 +- > > ovn/northd/ovn-northd.c | 941 +++++++++++++++++++++------- > > ovn/ovn-architecture.7.xml | 71 +++ > > ovn/ovn-nb.ovsschema | 42 +- > > ovn/ovn-nb.xml | 131 ++++ > > ovn/ovn-sb.ovsschema | 43 +- > > ovn/ovn-sb.xml | 63 ++ > > ovn/utilities/ovn-nbctl.8.xml | 41 ++ > > ovn/utilities/ovn-nbctl.c | 221 +++++++ > > ovn/utilities/ovn-sbctl.c | 6 + > > tests/ovn-northd.at | 568 ++++++++++++++++- > > tests/ovn.at | 1041 ++++++++++++++++++++++++++++++- > > 32 files changed, 3400 insertions(+), 861 deletions(-) > > delete mode 100644 ovn/controller/gchassis.c > > delete mode 100644 ovn/controller/gchassis.h > > create mode 100644 ovn/controller/ha-chassis.c > > create mode 100644 ovn/controller/ha-chassis.h > > > > -- > > 2.20.1 > > > > _______________________________________________ > > dev mailing list > > d...@openvswitch.org > > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > > Thanks Numan. For all patches in this series: > > Acked-by: Han Zhou <hzh...@ebay.com>
Sorry Numan, I forgot to mention a compiling error I saw with gcc: ../ovn/utilities/ovn-nbctl.c:721:77: error: string too long (8258 bytes, 8191 bytes max) There is no problem with clang. Not sure if there is problem in CI. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev