On Fri, Dec 18, 2020 at 1:16 AM Dumitru Ceara <[email protected]> wrote:

> 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 Ilya for adding this and Dumitru for the reviews.
I applied this patch to master.

Numan


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

Reply via email to