Hi Frode,

For patches 6-9:

Acked-by: Mark Michelson <[email protected]>

Can you specify which of the patches need to be backported and to which branches they need to be backported? I can take care of all the merges at the same time once I get this info.

Thanks.

On 3/5/21 7:16 AM, Frode Nordahl wrote:
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 <[email protected]>
---
  tests/ovn.at | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/ovn.at b/tests/ovn.at
index bec593dcc..ca9623fee 100644
--- a/tests/ovn.at
+++ b/tests/ovn.at
@@ -21572,7 +21572,7 @@ ovn-nbctl --wait=sb lsp-add ls1 lsp1
# Simulate the fact that lsp1 had been previously bound on hv1.
  ovn-sbctl --id=@e create encap chassis_name=hv1 ip="192.168.0.1" 
type="geneve" \
-    -- --id=@c create chassis name=hv1 encaps=@e \
+    -- --id=@c create chassis hostname=hv1 name=hv1 encaps=@e \
      -- set Port_Binding lsp1 chassis=@c
as hv1


_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to