On 7/29/20 9:43 AM, Numan Siddique wrote:


On Wed, Jul 29, 2020 at 1:54 AM Mark Michelson <mmich...@redhat.com <mailto:mmich...@redhat.com>> wrote:

    In some cases, it is important that traffic that arrived over an ECMP
    route has its replies sent out over that same route. This series of
    commits adds some prerequisite changes in order to facilitate using
    conntrack labels to store information about the incoming traffic. The
    final commit adds the new feature, storing the source MAC and incoming
    logical router port for new incoming connections over an ECMP route. The
    feature is only used if a specific option is set, and it may only be
    used in gateway routers.


Acked-by: Numan Siddique <num...@ovn.org <mailto:num...@ovn.org>> for the entire series.

Thanks
Numan

Thanks for the reviews Numan. I merged this to master.


    ---
    v3 -> v4
    * Rebased to current master
    * Fixed failing conntrack fields test
    * Bumped ovn-nb minor version.
    v2 -> v3
    * Added this cover letter
    * Added writeability scope commit to series
    * Added assignment writeability alteration commit to series
    * Added test to ovn.at <http://ovn.at> to ensure flows are only
    present on gateway
       routers
    * Changed the bits used in the ct_label. Previous commits used bits
       0-63. This commit changes it to use bits 32-95.
    v1 -> v2
    * Fixed test failures
    ---
    Mark Michelson (5):
       Add IP address normalization to ovn-utils.
       Don't check for writeability of rhs during assignment.
       Add expression writeability scopes.
       Used nested actions in ct_commit
       Add ECMP symmetric replies.

      include/ovn/actions.h     |   9 +-
      include/ovn/expr.h        |  53 +++++++---
      lib/actions.c             | 154 ++++++++--------------------
      lib/expr.c                |  35 ++++---
      lib/logical-fields.c      |  15 ++-
      lib/ovn-util.c            |  39 +++++++
      lib/ovn-util.h            |   4 +
      northd/ovn-northd.8.xml   |  49 +++++++--
      northd/ovn-northd.c       | 131 ++++++++++++++++++++----
      ovn-architecture.7.xml    |   7 +-
      ovn-nb.ovsschema          |   5 +-
      ovn-nb.xml                |  16 +++
      ovn-sb.xml                |  11 +-
      tests/ovn.at <http://ovn.at>              | 210
    +++++++++++++++++++++++++++++++-------
      tests/system-ovn.at <http://system-ovn.at>       | 144
    ++++++++++++++++++++++++++
      utilities/ovn-nbctl.8.xml |  31 ++++--
      utilities/ovn-nbctl.c     |  47 +++------
      17 files changed, 695 insertions(+), 265 deletions(-)

-- 2.25.4

    _______________________________________________
    dev mailing list
    d...@openvswitch.org <mailto: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

Reply via email to