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

promtool check metrics inconsistencies in "internal" metrics #250

Open
awelzel opened this issue Jul 5, 2022 · 1 comment
Open

promtool check metrics inconsistencies in "internal" metrics #250

awelzel opened this issue Jul 5, 2022 · 1 comment

Comments

@awelzel
Copy link
Contributor

awelzel commented Jul 5, 2022

Running promtool check metrics for the broker / caf generated Prometheus metrics indicates some conventions are not adhered to.

$ curl -sf http://localhost:4242/metrics | promtool check metrics
broker_processed_elements counter metrics should have "_total" suffix
caf_system_queued_messages_total non-counter metrics should not have "_total" suffix
process_cpu_seconds_total non-counter metrics should not have "_total" suffix

process_cpu_seconds_total could just be switched to a counter. Not sure about the semantics of caf_system_queued_messages_total (gauge vs counter). Might also be just a type difference.

@Neverlord
Copy link
Member

The metrics that come from CAF code (the process metrics also come from CAF) probably need to wait for CAF 0.19, the Broker metric I can update right away.

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

No branches or pull requests

2 participants