[ovs-git] [ovn-org/ovn] b92823: tests: Amend release stale port binding test for RBAC

Frode Nordahl noreply at github.com
Wed Mar 24 12:48:11 UTC 2021


  Branch: refs/heads/master
  Home:   https://github.com/ovn-org/ovn
  Commit: b92823f0e94e760c3e4b60ef132b513c3411ed2d
      https://github.com/ovn-org/ovn/commit/b92823f0e94e760c3e4b60ef132b513c3411ed2d
  Author: Frode Nordahl <frode.nordahl at canonical.com>
  Date:   2021-03-24 (Wed, 24 Mar 2021)

  Changed paths:
    M tests/ovn.at

  Log Message:
  -----------
  tests: Amend release stale port binding test for RBAC

The current version of the test attempts to simulate chassis
registration prior to starting `ovn-controller`, however it does
not set the `hostname` field.

The RBAC role for `ovn-controller` does not allow for a chassis to
change its own name or hostname, which makes sense as this is used
for authentication.

Update the test to set the `hostname` field when simulating chassis
registration so that `ovn-controller` does not attempt to update it
and subsequently make the test fail.

Fixes b6b3823d4 ("ovn-controller: Fix I-P for SB Port_Binding and OVS Interface")

Signed-off-by: Frode Nordahl <frode.nordahl at canonical.com>
Acked-by: Mark Michelson <mmichels at redhat.com>
Signed-off-by: Numan Siddique <numans at ovn.org>


  Commit: 020dab90f725b548a6131c988bd52e96623d3b8f
      https://github.com/ovn-org/ovn/commit/020dab90f725b548a6131c988bd52e96623d3b8f
  Author: Frode Nordahl <frode.nordahl at canonical.com>
  Date:   2021-03-24 (Wed, 24 Mar 2021)

  Changed paths:
    M tests/ovn-controller.at

  Log Message:
  -----------
  tests: Use ovn_start in tests/ovn-controller.at

The current version of the tests only initializes the SB DB and
instruments it directly. This does not work with SSL+RBAC as
northd must run to program the RBAC rules into the SB DB.

Run tests both for C and ddlog version of northd.

Add workaround for ovn-controller not re-reading certificates to
'ovn-controller - Chassis other_config' test.

Signed-off-by: Frode Nordahl <frode.nordahl at canonical.com>
Acked-by: Mark Michelson <mmichels at redhat.com>
Signed-off-by: Numan Siddique <numans at ovn.org>


Compare: https://github.com/ovn-org/ovn/compare/ff2b6ff69740...020dab90f725


More information about the git mailing list