Re: [ovs-dev] [PATCH ovn] binding.c: handle localnet port only if it is on local_datapaths.

2023-08-15 Thread Dumitru Ceara
On 8/14/23 00:31, Han Zhou wrote: > On Fri, Aug 11, 2023 at 11:30 PM Dumitru Ceara wrote: >> >> On Saturday, August 12, 2023, Han Zhou wrote: >>> >>> >>> >>> On Fri, Aug 11, 2023 at 5:36 AM Dumitru Ceara wrote: On 8/10/23 17:53, Han Zhou wrote: > Today the localnet ports are added

Re: [ovs-dev] [PATCH ovn] binding.c: handle localnet port only if it is on local_datapaths.

2023-08-13 Thread Han Zhou
On Fri, Aug 11, 2023 at 11:30 PM Dumitru Ceara wrote: > > On Saturday, August 12, 2023, Han Zhou wrote: >> >> >> >> On Fri, Aug 11, 2023 at 5:36 AM Dumitru Ceara wrote: >> > >> > On 8/10/23 17:53, Han Zhou wrote: >> > > Today the localnet ports are added to local_lports unconditionally. >> > > T

Re: [ovs-dev] [PATCH ovn] binding.c: handle localnet port only if it is on local_datapaths.

2023-08-11 Thread Dumitru Ceara
On Saturday, August 12, 2023, Han Zhou wrote: > > > On Fri, Aug 11, 2023 at 5:36 AM Dumitru Ceara wrote: > > > > On 8/10/23 17:53, Han Zhou wrote: > > > Today the localnet ports are added to local_lports unconditionally. > > > This has several side-effects: > > > > > > - When ovn-monitor-all is

Re: [ovs-dev] [PATCH ovn] binding.c: handle localnet port only if it is on local_datapaths.

2023-08-11 Thread Han Zhou
On Fri, Aug 11, 2023 at 5:36 AM Dumitru Ceara wrote: > > On 8/10/23 17:53, Han Zhou wrote: > > Today the localnet ports are added to local_lports unconditionally. > > This has several side-effects: > > > > - When ovn-monitor-all is true, all localnet ports are added to > > local_lports, includin

Re: [ovs-dev] [PATCH ovn] binding.c: handle localnet port only if it is on local_datapaths.

2023-08-11 Thread Dumitru Ceara
On 8/10/23 17:53, Han Zhou wrote: > Today the localnet ports are added to local_lports unconditionally. > This has several side-effects: > > - When ovn-monitor-all is true, all localnet ports are added to > local_lports, including the ones on the non-local logical switches, > which is undesira

[ovs-dev] [PATCH ovn] binding.c: handle localnet port only if it is on local_datapaths.

2023-08-10 Thread Han Zhou
Today the localnet ports are added to local_lports unconditionally. This has several side-effects: - When ovn-monitor-all is true, all localnet ports are added to local_lports, including the ones on the non-local logical switches, which is undesirable, although there is no direct consequences