On Mon, Mar 22, 2021 at 8:47 PM Mark Michelson <[email protected]> wrote:
>
> Hi Frode,
>
> For patches 6-9:
>
> Acked-by: Mark Michelson <[email protected]>
Thank you for the review, much appreciated.
> 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.
Numan already did the required backports for patches 1-5 so we should
be good there. Would be great to get the tests into master.
Cheers!
--
Frode Nordahl
> 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
