Re: [ovs-dev] [PATCH ovn 2/5] northd: Merge port group related structures.

2023-08-22 Thread Han Zhou
On Mon, Aug 21, 2023 at 11:23 PM Ales Musil wrote: > > > > On Thu, Aug 10, 2023 at 2:44 PM Dumitru Ceara wrote: >> >> There's currently no need to store both 'ovn_ls_port_group' >> and 'ovn_port_group_ls' so just use the first type. >> >> Signed-off-by: Dumitru Ceara > > > Hi Dumitru, > > I have

Re: [ovs-dev] [PATCH ovn 2/5] northd: Merge port group related structures.

2023-08-21 Thread Ales Musil
On Thu, Aug 10, 2023 at 2:44 PM Dumitru Ceara wrote: > There's currently no need to store both 'ovn_ls_port_group' > and 'ovn_port_group_ls' so just use the first type. > > Signed-off-by: Dumitru Ceara > Hi Dumitru, I have one small comment down below. --- > northd/en-lflow.c |2 > north

[ovs-dev] [PATCH ovn 2/5] northd: Merge port group related structures.

2023-08-10 Thread Dumitru Ceara
There's currently no need to store both 'ovn_ls_port_group' and 'ovn_port_group_ls' so just use the first type. Signed-off-by: Dumitru Ceara --- northd/en-lflow.c |2 northd/northd.c | 286 + northd/northd.h | 25 - 3 files chan