[ovs-dev] [PATCH] ovsdb: Add raft memory usage to memory report.

[Ilya Maximets]

On 5/22/20 10:36 PM, Han Zhou wrote:
> 
> 
> On Fri, May 22, 2020 at 10:36 AM Ilya Maximets <i.maximets at ovn.org <mailto:i.maximets at ovn.org>> wrote:
>>
>> Memory reports could be found in logs or by calling 'memory/show'
>> appctl command.  For ovsdb-server it includes information about db
>> cells, monitor connections with their backlog size, etc.  But it
>> doesn't contain any information about memory consumed by raft.
>> Backlogs of raft connections could be insanely large because of
>> snapshot installation requests that simply contains the whole database.
>> In not that healthy clusters where one of ovsdb servers is not able to
>> timely handle all the incoming raft traffic, backlog on a sender's side
>> could cause significant memory consumption issues.
>>
>> Adding new 'raft-connections' and 'raft-backlog' counters to the
>> memory report to better track such conditions.
>>
>> Signed-off-by: Ilya Maximets <i.maximets at ovn.org <mailto:i.maximets at ovn.org>>
>> ---
>>  ovsdb/ovsdb.c   |  4 ++++
>>  ovsdb/raft.c    | 16 ++++++++++++++++
>>  ovsdb/raft.h    |  2 ++
>>  ovsdb/storage.c | 10 ++++++++++
>>  ovsdb/storage.h |  3 +++
>>  5 files changed, 35 insertions(+)
>>
> Acked-by: Han Zhou <hzhou at ovn.org <mailto:hzhou at ovn.org>>

Applied to master. Thanks!

Best regards, Ilya Maximets.