[ovs-git] [ovn-org/ovn] ba0aa3: controller-vtep: fix mmr and physical locators cre...

Dumitru Ceara noreply at github.com
Tue Jun 15 20:43:25 UTC 2021


  Branch: refs/heads/branch-21.06
  Home:   https://github.com/ovn-org/ovn
  Commit: ba0aa3043f4755b05dedc3b51859306327f282c6
      https://github.com/ovn-org/ovn/commit/ba0aa3043f4755b05dedc3b51859306327f282c6
  Author: Vladislav Odintsov <odivlad at gmail.com>
  Date:   2021-06-15 (Tue, 15 Jun 2021)

  Changed paths:
    M controller-vtep/vtep.c
    M tests/ovn-controller-vtep.at

  Log Message:
  -----------
  controller-vtep: fix mmr and physical locators create/update

Before this patch ovn-controller-vtep created Mcast_Macs_Remote
record for each Port Binding in the ovn Logical Switch, to which
vtep Logical Switch was attached.
With this patch there is only one Mcast_Macs_Remote record per datapath.
Physical Locator set is created every time when physical locators for
associated datapath are changed. Next, this newly-created physical
locator set is updated in the MMR record.

Signed-off-by: Vladislav Odintsov <odivlad at gmail.com>
Acked-by: Dumitru Ceara <dceara at redhat.com>
Signed-off-by: Numan Siddique <numans at ovn.org>
(cherry picked from commit 8d95000952cc98d56d94be1cf5345fafa6531949)


  Commit: c95d3dd7970cb35f256e47611d0b69a8cafb30db
      https://github.com/ovn-org/ovn/commit/c95d3dd7970cb35f256e47611d0b69a8cafb30db
  Author: Vladislav Odintsov <odivlad at gmail.com>
  Date:   2021-06-15 (Tue, 15 Jun 2021)

  Changed paths:
    M controller-vtep/vtep.c

  Log Message:
  -----------
  controller-vtep: Update tunnel_key and replication_mode if needed

Signed-off-by: Vladislav Odintsov <odivlad at gmail.com>
Acked-by: Dumitru Ceara <dceara at redhat.com>
Signed-off-by: Numan Siddique <numans at ovn.org>
(cherry picked from commit aedf54c45db706c503ed976403ae41e954316bdf)


  Commit: 4b0a02635fde3d356851dddf12936d9ff2fcc874
      https://github.com/ovn-org/ovn/commit/4b0a02635fde3d356851dddf12936d9ff2fcc874
  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>
(cherry picked from commit 4f70baccb79fcb814b07cf1fe501ca47792c1e6c)


  Commit: 9d331e4f6c4f65861bd4e185d837a4b448bbedcf
      https://github.com/ovn-org/ovn/commit/9d331e4f6c4f65861bd4e185d837a4b448bbedcf
  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>
(cherry picked from commit c33395143507e60a453394d7c85da40d761ba0c1)


  Commit: fb16564b802b34851116b13cda442c05f3bb2966
      https://github.com/ovn-org/ovn/commit/fb16564b802b34851116b13cda442c05f3bb2966
  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>
(cherry picked from commit f6bd743cb9a7312cbaba2aa40ed746d3f1e4b8d9)


Compare: https://github.com/ovn-org/ovn/compare/f754318692e0...fb16564b802b


More information about the git mailing list