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 _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev