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

Han Zhou noreply at github.com
Fri Jun 11 22:29:58 UTC 2021


  Branch: refs/heads/branch-21.06
  Home:   https://github.com/ovn-org/ovn
  Commit: b013c944842a85496fb510a87f81fc95a77cdab2
      https://github.com/ovn-org/ovn/commit/b013c944842a85496fb510a87f81fc95a77cdab2
  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: acf865c0341c6b289739935106746ff26e101ab6
      https://github.com/ovn-org/ovn/commit/acf865c0341c6b289739935106746ff26e101ab6
  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: e87b2c0d567ced5eafd8a6bc733248461e650f46
      https://github.com/ovn-org/ovn/commit/e87b2c0d567ced5eafd8a6bc733248461e650f46
  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/153b1e43f1f1...e87b2c0d567c


More information about the git mailing list