[ovs-git] [ovn-org/ovn] 81c41c: ovn-northd.at: Fix test "northd ssl file change --...

Han Zhou noreply at github.com
Fri Jun 11 22:26:27 UTC 2021


  Branch: refs/heads/master
  Home:   https://github.com/ovn-org/ovn
  Commit: 81c41c3ec8a3d771161ae12e9e1aba54dbbf2132
      https://github.com/ovn-org/ovn/commit/81c41c3ec8a3d771161ae12e9e1aba54dbbf2132
  Author: Han Zhou <hzhou at ovn.org>
  Date:   2021-06-11 (Fri, 11 Jun 2021)

  Changed paths:
    M tests/ovn-northd.at

  Log Message:
  -----------
  ovn-northd.at: Fix test "northd ssl file change -- ovn-northd-ddlog".

This test fails for ovn-northd-ddlog because of the RBAC role when using
the SSL connection. RBAC is not the purpose of the test case, so this
patch fixes it without enabling RBAC.

Acked-by: Mark Michelson <mmichels at redhat.com>
Signed-off-by: Han Zhou <hzhou at ovn.org>


  Commit: 651211b65f09b86eec5ae05b874932303d7b8418
      https://github.com/ovn-org/ovn/commit/651211b65f09b86eec5ae05b874932303d7b8418
  Author: Han Zhou <hzhou at ovn.org>
  Date:   2021-06-11 (Fri, 11 Jun 2021)

  Changed paths:
    M tests/ovn.at

  Log Message:
  -----------
  ovn.at: Fix test "nb_cfg timestamp -- ovn-northd-ddlog".

The test case fails quite often because of timing. When nb_cfg_timestamp
is updated in SB DB it may not be updated immediately in NB's
hv_cfg_timestamp. This patch use "wait_column" to wait.

Acked-by: Mark Michelson <mmichels at redhat.com>
Signed-off-by: Han Zhou <hzhou at ovn.org>


  Commit: f9eb118364f0efc9452c7235ca566000b110767a
      https://github.com/ovn-org/ovn/commit/f9eb118364f0efc9452c7235ca566000b110767a
  Author: Han Zhou <hzhou at ovn.org>
  Date:   2021-06-11 (Fri, 11 Jun 2021)

  Changed paths:
    M tests/ovn.at

  Log Message:
  -----------
  ovn.at: Fix test "No ovn-controller assert when generating conjunction flows -- ovn-northd-ddlog".

The test fails quite often for ddlog because of tunnel keys mismatch when
checking OpenFlow dumps. It is possible to allocate different keys in
different runs because multiple NB changes can be handled by northd in a
single iteration - it fails more often in ddlog probably because there
is some more latency for northd-ddlog to start processing the changes,
but it doesn't affect correctness. So this patch fixes it by ignoring
the tunnel keys when comparing the flows.

Acked-by: Mark Michelson <mmichels at redhat.com>
Signed-off-by: Han Zhou <hzhou at ovn.org>


Compare: https://github.com/ovn-org/ovn/compare/e230fa009b81...f9eb118364f0


More information about the git mailing list