search

sensu-plugins / sensu-plugins-memory-checks

This plugin provides native memory instrumentation for monitoring and metrics collection, including: memory usage via `free` and `vmstat`, including metrics. Note that this plugin may have cross-platform issues.
http://sensu-plugins.io
MIT License
15 stars 50 forks source link

Change source of data in check-ram - closes #14 #15

Closed 11mariom closed 8 years ago

11mariom commented 9 years ago

Hi,

I edited check to fix issue I reported (#14). It works on Linux very well. I tried using lib for vmstat - but I cannot get with it values of buffers/cache ;(

mattyjones commented 9 years ago

@11mariom I will take a look at this but I will say I am not inclined to use proc unless absolutely necessary.

@tas50 you agree ?

11mariom commented 9 years ago

@mattyjones well - I had to do something to fix failing nodes. I know it is not best way to do it, but... it is better than using free -m | grep buffers - as it will not fail on CentOS 7

mattyjones commented 9 years ago

@11mariom Can you rebase this and I will merge

Thanks

11mariom commented 9 years ago

@mattyjones what do you want me to rebase?

analytically commented 9 years ago

I think he meant rebase the PR.

eheydrick commented 8 years ago

Bump for rebase.

majormoses commented 8 years ago

I think this can be closed as the vmstat rewrite has landed.

eheydrick commented 8 years ago

Closing as check-ram uses vmstat now.