On Fri, Aug 11, 2023 at 5:36 AM Dumitru Ceara <dce...@redhat.com> 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, including the ones on the non-local logical switches, > > which is undesirable, although there is no direct consequences yet. > > > > - When ovn-monitor-all is false (i.e. conditional monitoring is > > enabled), only localnet ports of local datapaths are supposed to be > > monitored. However, because of the issue discussed at [0], conditional > > monitoring doesn't take effect initially, and all the localnet ports > > are monitored and added to local_lports. Later when monitor condition > > is set for port_bindings, all the localnet ports are added to the > > condition, which causes scale problem e.g. in ovn-k8s environment > > where each node has at least one localnet port, and the SB DB servers > > are overloaded when handling such huge conditions from every node. > > > > [0] https://mail.openvswitch.org/pipermail/ovs-dev/2023-July/406201.html > > > > Signed-off-by: Han Zhou <hz...@ovn.org> > > --- > > Looks good to me, thanks! > > Acked-by: Dumitru Ceara <dce...@redhat.com> > > Regards, > Dumitru >
Thanks Dumitru. I applied the patch to main. Regards, Han _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
