[ovs-git] [ovn-org/ovn] 4f70ba: ovs: Include ovs-vswitchd segfault fixes.

[Dumitru Ceara]

  Branch: refs/heads/master
  Home:   https://github.com/ovn-org/ovn
  Commit: 4f70baccb79fcb814b07cf1fe501ca47792c1e6c
      https://github.com/ovn-org/ovn/commit/4f70baccb79fcb814b07cf1fe501ca47792c1e6c
  Author: Dumitru Ceara <dceara at redhat.com>
  Date:   2021-06-15 (Tue, 15 Jun 2021)

  Changed paths:
    M ovs

  Log Message:
  -----------
  ovs: Include ovs-vswitchd segfault fixes.

Bump the OVS submodule to include the following patches:
  dd0f59783e39 ("ofproto: Fix potential NULL dereference in ofproto_get_datapath_cap().")
  a4b04276ab59 ("ofproto: Fix potential NULL dereference in ofproto_ct_*_zone_timeout_policy().")

The two commits above fix crashes in ovs-vswitchd when OVS datapath
types are configured with unsupported values and OVSDB Datapath tables
exist for those datapath types.  OVN self tests actually test with
invalid datapath types and an upcoming commit will update the
ovn-controller code to always create OVSDB Datapath tables.  Without
the two OVS fixes, ovs-vswitchd would crash, causing the OVN tests to
fail.

Acked-by: Mark D. Gray <mark.d.gray at redhat.com>
Signed-off-by: Dumitru Ceara <dceara at redhat.com>
Signed-off-by: Numan Siddique <numans at ovn.org>


  Commit: c33395143507e60a453394d7c85da40d761ba0c1
      https://github.com/ovn-org/ovn/commit/c33395143507e60a453394d7c85da40d761ba0c1
  Author: Dumitru Ceara <dceara at redhat.com>
  Date:   2021-06-15 (Tue, 15 Jun 2021)

  Changed paths:
    M tests/system-ovn.at

  Log Message:
  -----------
  system-ovn.at: Use ADD_BR macro instead of bare ovs-vsctl.

br-int and br-ex are patched together by ovn-controller.  They must have
the same datapath type.  I.e., it won't work if br-int uses the netdev
(userspace) datapath and br-ext uses the system (kernel) datapath;
system is the default.  This ensures the datapath type is properly set
for the br-ext bridge.

a8362ff85dee ("northd: Fix the missing force_snat_for_lb flows when router_ip is configured.")
Acked-by: Mark D. Gray <mark.d.gray at redhat.com>
Signed-off-by: Dumitru Ceara <dceara at redhat.com>
Signed-off-by: Numan Siddique <numans at ovn.org>


  Commit: f6bd743cb9a7312cbaba2aa40ed746d3f1e4b8d9
      https://github.com/ovn-org/ovn/commit/f6bd743cb9a7312cbaba2aa40ed746d3f1e4b8d9
  Author: Dumitru Ceara <dceara at redhat.com>
  Date:   2021-06-15 (Tue, 15 Jun 2021)

  Changed paths:
    M .github/workflows/test.yml

  Log Message:
  -----------
  github: Use nmap-ncat instead of netcat-openbsd.

There are some bugs in the netcat-openbsd version shipped with Ubuntu
20.04.  Switch to nmap-ncat to avoid system tests that will be added
by upcoming commits fail in CI.

One example:

1. Start a TCP connection from IP1:port-x to IP2:port-y.
2. Start a TCP connection from IP1:port-x to IP3:port-z.

netcat-openbsd fails with:
  nc: bind failed: Address already in use

Acked-by: Mark D. Gray <mark.d.gray at redhat.com>
Signed-off-by: Dumitru Ceara <dceara at redhat.com>
Signed-off-by: Numan Siddique <numans at ovn.org>


Compare: https://github.com/ovn-org/ovn/compare/aedf54c45db7...f6bd743cb9a7