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

additional metrics #29

Closed
EvertMDC opened this issue Aug 24, 2016 · 6 comments
Closed

additional metrics #29

EvertMDC opened this issue Aug 24, 2016 · 6 comments

Comments

@EvertMDC
Copy link

Hello,

Would there be a possibility to add the following metrics to the exported data?

consul.consul.leader.reconcile.count
consul.serf.events.consul_new_leader

It's based on this article I found regarding consul monitoring.
It does makes sense

@jeinwag
Copy link
Contributor

jeinwag commented Sep 19, 2016

Unfortunately, these metrics aren't exposed via the Consul HTTP or RPC APIs.

Currently, the only way to get them into Prometheus is using the statsd_exporter.

@grobie
Copy link
Member

grobie commented Oct 20, 2016

There is a long standing issue for providing all metrics over a HTTP API: hashicorp/consul#817.

At SoundCloud we have a side-car next to every consul agent which queries serf and other stats via the consul ctl / rpc interface and exposes it via Prometheus.

@EvertMDC
Copy link
Author

Some consistency would indeed be good.
We are currently using the statsd exporter running next to consul.

@arianvp
Copy link

arianvp commented Dec 18, 2017

This has been resolved in hashicorp/consul#3369

More metrics are now exposed through the HTTP API

@Luubosi
Copy link

Luubosi commented Oct 21, 2018

hi,
will be metrics added from /v1/agent/metrics?
Could you tell me some date?

or if it is possible only with statsd, do you have YAML-based config format of this config
https://gist.github.com/dj-wasabi/d9b31c4b74e561c72512f4edbdfe6927
thanks

@simonpasquier
Copy link
Member

Recent versions of Consul support the Prometheus exposition natively (see https://www.consul.io/docs/agent/options#telemetry-prometheus_retention_time) so this doesn't need to be reimplemented in the exporter IMHO.

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

Successfully merging a pull request may close this issue.

7 participants