[ovs-git] [ovn-org/ovn] b3feef: Fix selection fields for UDP and SCTP load balancers.

numansiddique noreply at github.com
Tue Jun 30 15:36:03 UTC 2020


  Branch: refs/heads/master
  Home:   https://github.com/ovn-org/ovn
  Commit: b3feef2a774d536f0ca20d15fe8e9c9274d0b6bd
      https://github.com/ovn-org/ovn/commit/b3feef2a774d536f0ca20d15fe8e9c9274d0b6bd
  Author: Numan Siddique <numans at ovn.org>
  Date:   2020-06-30 (Tue, 30 Jun 2020)

  Changed paths:
    M northd/ovn-northd.c
    M tests/ovn.at
    M tests/system-ovn.at

  Log Message:
  -----------
  Fix selection fields for UDP and SCTP load balancers.

The commit 5af304e7478a ("Support selection fields in load balancer.")
didn't handle the selection fields for UDP and SCTP protocol.
If CMS has set the selection fields - tp_src and tp_dst for UDP or SCTP
load balancers, ovn-northd was adding lflows as
ct_lb(backends=<backends>, hash_fields(tp_src,tp_dst))
instead of ct_lb(backends=<backends>, hash_fields(udp_src,udp_dst)) and
ct_lb(backends=<backends>, hash_fields(sctp_src,sctp_dst)) respectively.

Because of this, select group flows were encoded with tcp_src and tcp_dst
hash fields.

This patch fixes this issue.

Test cases are now added for UDP and SCTP protocols.

Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=1846189
Signed-off-by: Numan Siddique <numans at ovn.org>
Acked-by: Mark Michelson <mmichels at redhat.com>




More information about the git mailing list