[ovs-dev] ovn-controller doesn't bring re-added container ports up?

[Numan Siddique]

On Tue, Feb 2, 2021 at 11:26 PM Numan Siddique <numans at ovn.org> wrote:
>
> On Tue, Feb 2, 2021 at 11:11 PM Ben Pfaff <blp at ovn.org> wrote:
> >
> > The test "ovn -- nested containers" adds, removes, and then re-adds a
> > container port bar2.  It appears to me that ovn-controller never brings
> > the re-added port up.  I don't know why.
> >
> > To see the problem, apply the following patch then run that test
> > (currently test 88):
>
> Hi Ben,
>
> Thanks for reporting this. I was able to reproduce locally. Let me debug
> further and get back to you.
>

This regression is seen after the commit [1].

Dumitru submitted a patch today [2] to fix an issue introduced by [1]
and looks like his patch
doesn't address this particular issue.

@Dumitru - Can you also please address the issue reported here in your patch.

[1] - 4d3cb42b07 ("binding: Set Logical_Switch_Port.up when all OVS
flows are installed.")
[2] - https://patchwork.ozlabs.org/project/ovn/patch/1612259114-30311-1-git-send-email-dceara@redhat.com/

Thanks
Numan

> Thanks
> Numan
>
> >
> > diff --git a/tests/ovn.at b/tests/ovn.at
> > index e011263838d4..b1e04ac16b4c 100644
> > --- a/tests/ovn.at
> > +++ b/tests/ovn.at
> > @@ -8988,9 +8988,9 @@ bar2_zoneid=$(as hv2 ovs-vsctl get bridge br-int external_ids:ct-zone-bar2)
> >  AT_CHECK([test  -z $bar2_zoneid])
> >
> >  # Add back bar2
> > -wait_for_ports_up
> >  ovn-nbctl lsp-add bar bar2 vm2 1 \
> >  -- lsp-set-addresses bar2 "f0:00:00:01:02:08 192.168.2.3"
> > +wait_for_ports_up
> >  ovn-nbctl --wait=hv sync
> >
> >  bar2_zoneid=$(as hv2 ovs-vsctl get bridge br-int external_ids:ct-zone-bar2)
> > _______________________________________________
> > dev mailing list
> > dev at openvswitch.org
> > https://mail.openvswitch.org/mailman/listinfo/ovs-dev
> >