search

munin-monitoring / munin

Main repository for munin master / node / plugins
http://munin-monitoring.org
Other
1.99k stars 474 forks source link

Incorrect Cached memory calculation #1591

Open starinacool opened 1 year ago

starinacool commented 1 year ago

We should not decrease cached memory by amount of shared memory. We should exclude shared memory from apps memory.

https://github.com/munin-monitoring/munin/blob/16eff1e959e1ed3dbdb615c3834a2a596efebcc9/plugins/node.d.linux/memory#L304C80-L304C80

steveschnepp commented 1 year ago

Thanks!

Do you have some documentation that shows our implementation is incorrect ?

steveschnepp commented 1 year ago

You already replied in https://github.com/munin-monitoring/munin/pull/1592#issuecomment-1801190601 👍

BETLOG commented 11 months ago

I was about to ask a similar question. Is this issue whats causing this slightly horrifying graph escalation? 2023-11-19--14-08-10_betlogbeast_memoryMuninmuninbetlogbeastmemoryGoogleChrome

steveschnepp commented 11 months ago

No. that is CommittedAS which is increasing too much. And it seems to be due to some change between the kernel version 5.10 and 6.1, as a debian 11 to 12 upgrade shows it.