On 12/17/20 7:14 PM, Ilya Maximets wrote:
> It is important to have memory reports in logs to track the state
> of OVN daemons.  With this change we will see the dynamic of the
> memory consumption.
> 
> We have no any specific information to report yet, but we have to
> call memory_report() to reply to all the incoming 'memory/show'
> requests.
> 
> Not documenting 'memory/show' command as there is nothing to
> report there yet.  This change will only add reports about
> memory usage growth to logs, e.g.:
> 
>   |memory|INFO|peak resident set size grew 50% in last 8 seconds,
>                from 27548 kB to 41336 kB
> 
> Signed-off-by: Ilya Maximets <[email protected]>
> ---

A tiny question: Do you already have ideas about what specific memory
usage reports we should add?  If so, should we add an item to TODO.rst
to address this in the near future?

In any case, it's already useful to get some memory usage information:

Acked-by: Dumitru Ceara <[email protected]>

Thanks,
Dumitru

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to