Re: [ovs-dev] [PATCH ovn v3 2/5] binding.c: Missing local_datapath update in runtime_data port_binding handler.

2022-08-19 Thread Han Zhou
On Fri, Aug 19, 2022 at 12:40 PM Dumitru Ceara wrote: > > On 8/19/22 17:35, Han Zhou wrote: > > On Fri, Aug 19, 2022 at 3:06 AM Dumitru Ceara wrote: > >> > >> On 8/19/22 11:10, Numan Siddique wrote: > >>> On Fri, Aug 19, 2022 at 10:42 AM Han Zhou wrote: > > On Thu, Aug 18, 2022 at

Re: [ovs-dev] [PATCH ovn v3 2/5] binding.c: Missing local_datapath update in runtime_data port_binding handler.

2022-08-19 Thread Dumitru Ceara
On 8/19/22 17:35, Han Zhou wrote: > On Fri, Aug 19, 2022 at 3:06 AM Dumitru Ceara wrote: >> >> On 8/19/22 11:10, Numan Siddique wrote: >>> On Fri, Aug 19, 2022 at 10:42 AM Han Zhou wrote: On Thu, Aug 18, 2022 at 5:21 PM Han Zhou wrote: > > When handling port_binding changes,

Re: [ovs-dev] [PATCH ovn v3 2/5] binding.c: Missing local_datapath update in runtime_data port_binding handler.

2022-08-19 Thread Han Zhou
On Fri, Aug 19, 2022 at 3:06 AM Dumitru Ceara wrote: > > On 8/19/22 11:10, Numan Siddique wrote: > > On Fri, Aug 19, 2022 at 10:42 AM Han Zhou wrote: > >> > >> On Thu, Aug 18, 2022 at 5:21 PM Han Zhou wrote: > >>> > >>> When handling port_binding changes, it is possible that new > >>>

Re: [ovs-dev] [PATCH ovn v3 2/5] binding.c: Missing local_datapath update in runtime_data port_binding handler.

2022-08-19 Thread Dumitru Ceara
On 8/19/22 11:10, Numan Siddique wrote: > On Fri, Aug 19, 2022 at 10:42 AM Han Zhou wrote: >> >> On Thu, Aug 18, 2022 at 5:21 PM Han Zhou wrote: >>> >>> When handling port_binding changes, it is possible that new >>> local_datapaths are added, and the fields of the local_datapath, such as >>>

Re: [ovs-dev] [PATCH ovn v3 2/5] binding.c: Missing local_datapath update in runtime_data port_binding handler.

2022-08-19 Thread Numan Siddique
On Fri, Aug 19, 2022 at 10:42 AM Han Zhou wrote: > > On Thu, Aug 18, 2022 at 5:21 PM Han Zhou wrote: > > > > When handling port_binding changes, it is possible that new > > local_datapaths are added, and the fields of the local_datapath, such as > > localnet_port, external_ports, etc. need to be

Re: [ovs-dev] [PATCH ovn v3 2/5] binding.c: Missing local_datapath update in runtime_data port_binding handler.

2022-08-18 Thread Han Zhou
On Thu, Aug 18, 2022 at 5:21 PM Han Zhou wrote: > > When handling port_binding changes, it is possible that new > local_datapaths are added, and the fields of the local_datapath, such as > localnet_port, external_ports, etc. need to be updated for the newly > added local_datapaths. > > This

[ovs-dev] [PATCH ovn v3 2/5] binding.c: Missing local_datapath update in runtime_data port_binding handler.

2022-08-18 Thread Han Zhou
When handling port_binding changes, it is possible that new local_datapaths are added, and the fields of the local_datapath, such as localnet_port, external_ports, etc. need to be updated for the newly added local_datapaths. This problem doesn't happen in most cases because the changes that