[ovs-dev] [OVN[ Backporting the commit e07e397b7a("ovn-controller: Split logical flow and physical flow processing.") to branch-21.06
Numan Siddique
numans at ovn.org
Tue Jul 27 18:24:49 UTC 2021
On Tue, Jul 27, 2021 at 1:21 AM Han Zhou <hzhou at ovn.org> wrote:
>
> On Mon, Jul 26, 2021 at 7:52 PM Numan Siddique <numans at ovn.org> wrote:
> >
> > We have a bug in the present branch-21.06. Unfortunately the commit
> > e1e78b4b148("binding: Fix the crashes seen when port binding type
> > changes.")
> > introduced a regression for the localports. When an OVS interface for
> > a localport is created by CMS, ovn-controller
> > doesn't update or add the existing openflows in table 38 for multicast
> > groups if the localport belongs to a multicast group.
> > This causes the broadcast traffic destined to localport to drop and it
> > breaks the openstack metadata service.
> > The issue goes away when the ovn-controller claims subsequent ports
> > which belong to the same logical switch or
> > if there is a full recompute.
> >
> > The commit e07e397b7a which split the logical flows and physical flow
> > handling fixes this issue.
> >
> > Is it OK to backport this commit to branch-21.06 ? Please let me know
> > if it seems risky to backport.
> > In which case I'll submit a patch targeting the branch-21.06 to
> > address this issue.
> >
> > I prefer the backport because it seems cleaner to me.
> >
> > Let me know what you think.
> >
> > Regards
> > Numan
>
> Hi Numan,
>
> I am ok with backporting.
Thanks. I backported that patch to branch-21.06.
Numan
>
> Thanks,
> Han
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
More information about the dev
mailing list