[ovs-git] [ovn-org/ovn] 48d8b0: tests: fix flaky VXLAN check port/datapath key spa...

[Dumitru Ceara]

  Branch: refs/heads/main
  Home:   https://github.com/ovn-org/ovn
  Commit: 48d8b07043300d3cca13762da984c56841f353b5
      https://github.com/ovn-org/ovn/commit/48d8b07043300d3cca13762da984c56841f353b5
  Author: Vladislav Odintsov <odivlad at gmail.com>
  Date:   2021-11-26 (Fri, 26 Nov 2021)

  Changed paths:
    M tests/ovn.at

  Log Message:
  -----------
  tests: fix flaky VXLAN check port/datapath key space limits test

OVN requested-tnl-key VXLAN checks affect tunnel_key in
datapath_binding record only on datapath tunnel key assigning.
If a dp tnl key was already assigned and VXLAN-enabled chassis
is added after that, the value would never be changed.

The test didn't check the VXLAN-enabled chassis was created and
thus test could fail like this:

$ ovn.at:3777: waiting until test x`ovn-nbctl lsp-get-up lsp-bad` = xup...
$ ovn.at:3777: wait succeeded immediately
$ ./ovn.at:3777: ovn-sbctl get Datapath_Binding ls-bad tunnel_key
$ --- -	2021-11-23 17:34:51.646403185 +0300
$ +++ /builddir/build/BUILD/ovn-21.09.90/tests/testsuite.dir/at-groups/104/stdout	2021-11-23 17:34:51.644406189 +0300
$ @@ -1,2 +1,2 @@
$ -1
$ +5000

This patch fixes such fail by adding await for chassis to be
registered in SB DB.

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>


  Commit: acffcc0dde6c641aeaf516e0b600813c7bc4d489
      https://github.com/ovn-org/ovn/commit/acffcc0dde6c641aeaf516e0b600813c7bc4d489
  Author: Dumitru Ceara <dceara at redhat.com>
  Date:   2021-11-26 (Fri, 26 Nov 2021)

  Changed paths:
    M NEWS
    M northd/northd.c
    M northd/ovn-northd.8.xml
    M ovn-nb.xml
    M tests/ovn-northd.at

  Log Message:
  -----------
  nb: Add support for gateway_mtu_bypass.

There are various costs (e.g., not being able to perform hardware
offload in some cases) when using check_pkt_larger() so the CMS
can now limit the impact by bypassing the packet length checks for
specific types of traffic (e.g., TCP).

Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2011779
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/602896502fcb...acffcc0dde6c