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

Check prometheus metrics #11

Closed
MarcoGomes00 opened this issue Jan 16, 2019 · 4 comments
Closed

Check prometheus metrics #11

MarcoGomes00 opened this issue Jan 16, 2019 · 4 comments

Comments

@MarcoGomes00
Copy link

MarcoGomes00 commented Jan 16, 2019

When I test the metrics form exporter I'm getting this result

curl -qs 0.0.0.0:9479/metrics | ./promtool check metrics
filebeat_cpu_ticks counter metrics should have "_total" suffix
filebeat_cpu_time_miliseconds counter metrics should have "_total" suffix
filebeat_libbeat_config_reloads counter metrics should have "_total" suffix
filebeat_libbeat_output counter metrics should have "_total" suffix
filebeat_libbeat_output_read_bytes counter metrics should have "_total" suffix
filebeat_libbeat_output_read_errors counter metrics should have "_total" suffix
filebeat_libbeat_output_write_bytes counter metrics should have "_total" suffix
filebeat_libbeat_output_write_errors counter metrics should have "_total" suffix
filebeat_memstats_gc_next counter metrics should have "_total" suffix
filebeat_memstats_memory_total non-counter metrics should not have "_total" suffix
filebeat_system_cpu_cores counter metrics should have "_total" suffix
filebeat_uptime_seconds counter metrics should have "_total" suffix

Can you please check?

@shivas
Copy link
Contributor

shivas commented Feb 7, 2019

Thanks for the report, yes makes sense, but given that this exporter is just mapping of JSON to Prometheus metrics I'm not sure changing metrics names would create more clarity in this case.

@lichuan0620
Copy link

Same here. For me, adding the _total suffix makes it more clear what kind of metrics I am working with.

@shivas shivas closed this as completed in 067e112 Oct 3, 2019
@CNG
Copy link

CNG commented Aug 4, 2020

I think filebeat_memstats_gc_next was correct as that is a gauge, not a counter?

How to interpret memory stats in filebeat log - Elastic Stack / Beats - Discuss the Elastic Stack

// NextGC is the target heap size of the next GC cycle.
//
// The garbage collector's goal is to keep HeapAlloc ≤ NextGC.
// At the end of each GC cycle, the target for the next cycle
// is computed based on the amount of reachable data and the
// value of GOGC.

@CNG
Copy link

CNG commented Aug 4, 2020

Same with CPU cores, and maybe uptime seconds? It's true uptime seconds counts up but it's a constant rate, you wouldn't take a rate() over it, it's more like a gauge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants