[ovs-dev] [PATCH v6 0/5] ovn: Add HA chassis group and 'external' port

[Han Zhou]

On Wed, Mar 27, 2019 at 11:34 AM <nusiddiq at redhat.com> wrote:
>
> From: Numan Siddique <nusiddiq at 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
> dev at openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Thanks Numan. For all patches in this series:

Acked-by: Han Zhou <hzhou8 at ebay.com>