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

Reply via email to