Re: [ovs-dev] [PATCH ovn] controller: physical: fix regression for container ports
On Fri, Jul 29, 2022 at 12:04 AM Ihar Hrachyshka wrote: > > Oops, sorry about that. This should be backported to 22.06. > > Acked-By: Ihar Hrachyshka > On Thu, Jul 28, 2022 at 9:55 AM Lorenzo Bianconi > 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 > > --- > > 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(, dp_key, port_key); > > +match_outport_dp_and_port_keys(, dp_key, > > parent_pb->tunnel_key); > > match_set_reg_masked(, 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
Re: [ovs-dev] [PATCH ovn] controller: physical: fix regression for container ports
Oops, sorry about that. This should be backported to 22.06. Acked-By: Ihar Hrachyshka 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 > --- > 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(, dp_key, port_key); > +match_outport_dp_and_port_keys(, dp_key, > parent_pb->tunnel_key); > match_set_reg_masked(, 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
[ovs-dev] [PATCH ovn] controller: physical: fix regression for container ports
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 --- 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(, dp_key, port_key); +match_outport_dp_and_port_keys(, dp_key, parent_pb->tunnel_key); match_set_reg_masked(, 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