Re: [ovs-dev] [PATCH ovn v3 1/3] ovn-northd: Remove lflow_add_unique.

2021-06-29 Thread Han Zhou
On Tue, Jun 29, 2021 at 7:04 AM Dumitru Ceara  wrote:
>
> On 6/21/21 8:51 AM, Han Zhou wrote:
> > This patch removes the workaround when adding multicast group related
> > lflows, because the multicast group dependency problem is fixed in
> > ovn-controller in the previous commit.
> >
> > This patch also removes the UniqueFlow/AnnotatedFlow usage in northd
> > DDlog implementation for the same reason.
> >
> > Signed-off-by: Han Zhou 
> > ---
>
> Hi Han,
>
> The changes look good to me.
>
> Acked-by: Dumitru Ceara 
>
> Thanks,
> Dumitru
>

Thanks Dumitru. I applied patch 1/3 - 2/3 of the series to master. I will
update the test case of patch 3/3 with v4.
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


Re: [ovs-dev] [PATCH ovn v3 1/3] ovn-northd: Remove lflow_add_unique.

2021-06-29 Thread Dumitru Ceara
On 6/21/21 8:51 AM, Han Zhou wrote:
> This patch removes the workaround when adding multicast group related
> lflows, because the multicast group dependency problem is fixed in
> ovn-controller in the previous commit.
> 
> This patch also removes the UniqueFlow/AnnotatedFlow usage in northd
> DDlog implementation for the same reason.
> 
> Signed-off-by: Han Zhou 
> ---

Hi Han,

The changes look good to me.

Acked-by: Dumitru Ceara 

Thanks,
Dumitru

___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev