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

numansiddique noreply at github.com
Tue Jun 30 15:42:50 UTC 2020


  Branch: refs/heads/branch-20.06
  Home:   https://github.com/ovn-org/ovn
  Commit: 2640b73ac7edb717331ff5256484a974d3473038
      https://github.com/ovn-org/ovn/commit/2640b73ac7edb717331ff5256484a974d3473038
  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>

(cherry-picked from master commit b3feef2a774d536f0ca20d15fe8e9c9274d0b6bd)




More information about the git mailing list