Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[metricbeats] Enrich statistics about swap and memory #13744

Closed
unixsurfer opened this issue Sep 20, 2019 · 3 comments
Closed

[metricbeats] Enrich statistics about swap and memory #13744

unixsurfer opened this issue Sep 20, 2019 · 3 comments
Labels
enhancement Metricbeat Metricbeat Team:Integrations Label for the Integrations team

Comments

@unixsurfer
Copy link

unixsurfer commented Sep 20, 2019

Describe the enhancement:
Add the following metrics to the list of exposed memory/swap metrics
pgscank = pgscan_kswapd from /proc/vmstat, number of pages scanned by the kswapd daemon
pgscand = pgscan_direct from /proc/vmstat, number of pages scanned directly
pgfreed = pgfree from /proc/vmstat, number of pages freed by the system
pgsteal = pgsteal from /proc/vmstat, Number of pages placed on the free list by the system
We call GetVMStat to fetch a VMStatInfo struct but we only fetch few metrics, while GetVMStat returns the aforementioned struct which has all metrics from /proc/vmstat.

Moreover, it would be great if you can caluclate the vmeff metric that sar tool list:

  %vmeff
                     Calculated as pgsteal / pgscan, this is a metric of the efficiency  of  page
                     reclaim.  If  it  is near 100% then almost every page coming off the tail of
                     the inactive list is being reaped. If it gets too low (e.g. less  than  30%)
                     then  the virtual memory is having some difficulty.  This field is displayed
                     as zero if no pages have been scanned during the interval of time.

Describe a specific use case for the enhancement or feature:
These metrics will help us to have visibility in memory saturation cases

@unixsurfer unixsurfer added enhancement Metricbeat Metricbeat Team:Integrations Label for the Integrations team labels Sep 20, 2019
@unixsurfer
Copy link
Author

Ideally, it would be great to expose everything that VMStatInfo struct contains.

@fearful-symmetry
Copy link
Contributor

@unixsurfer these changes have been merged and should be in 7.6. I'm gonna close this, if there's any related changes can we open it in another issue?

@unixsurfer
Copy link
Author

@fearful-symmetry Thanks a lot for work on this. It is very much appreciated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Metricbeat Metricbeat Team:Integrations Label for the Integrations team
Projects
None yet
Development

No branches or pull requests

3 participants