On Fri, Jul 29, 2022 at 12:04 AM Ihar Hrachyshka <ihrac...@redhat.com> wrote:
>
> Oops, sorry about that. This should be backported to 22.06.
>
> Acked-By: Ihar Hrachyshka <ihrac...@redhat.com

Hi Lorenzo,

Thanks for the fix.  Is it possible to add the test case to avoid
regression in the future for this ?

Looks like the reported BZ already has a reproducer.

Numan

>
> On Thu, Jul 28, 2022 at 9:55 AM Lorenzo Bianconi
> <lorenzo.bianc...@redhat.com> wrote:
> >
> > After commit 'd07e5f99d ("Introduce match_outport_dp_and_port_keys
> > in physical.c")' it is not longer possible to ping a parent port from
> > container one.
> >
> > Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2105901
> > Fixes: d07e5f99d ("Introduce match_outport_dp_and_port_keys in physical.c")
> > Signed-off-by: Lorenzo Bianconi <lorenzo.bianc...@redhat.com>
> > ---
> >  controller/physical.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/controller/physical.c b/controller/physical.c
> > index 816a557e7..a92534a03 100644
> > --- a/controller/physical.c
> > +++ b/controller/physical.c
> > @@ -929,7 +929,7 @@ put_local_common_flows(uint32_t dp_key,
> >           * unnecessary.
> >           */
> >          ofpbuf_clear(ofpacts_p);
> > -        match_outport_dp_and_port_keys(&match, dp_key, port_key);
> > +        match_outport_dp_and_port_keys(&match, dp_key, 
> > parent_pb->tunnel_key);
> >          match_set_reg_masked(&match, MFF_LOG_FLAGS - MFF_REG0,
> >                               MLF_NESTED_CONTAINER, MLF_NESTED_CONTAINER);
> >
> > --
> > 2.37.1
> >
>
> _______________________________________________
> dev mailing list
> d...@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to