Metrics missing from documentation

[maxb]

trafficstars

https://www.consul.io/docs/agent/telemetry#metrics-reference claims "This is a full list of metrics emitted by Consul." but some are missing:

• The family of new consul.acl.* metrics defined at https://github.com/hashicorp/consul/blob/019aeaa57de97100d57f8ab9c771365b1cfcf7b8/agent/consul/acl_endpoint.go#L33

• consul.consul.cache.entries_count defined at https://github.com/hashicorp/consul/blob/019aeaa57de97100d57f8ab9c771365b1cfcf7b8/agent/cache/cache.go#L38-L44 (the doubled initial consul. is presumably what the TODO comment is talking about - a number of other metrics are documented without the doubled consul. even though they are actually emitted with it - but that's a separate issue.)

• consul.federation_state.* from https://github.com/hashicorp/consul/blob/019aeaa57de97100d57f8ab9c771365b1cfcf7b8/agent/consul/federation_state_endpoint.go#L16-L33

• consul.intention.apply from https://github.com/hashicorp/consul/blob/019aeaa57de97100d57f8ab9c771365b1cfcf7b8/agent/consul/intention_endpoint.go#L88

• consul.raft.peers from within the github.com/hashicorp/raft library

• consul.runtime.gc_pause_ns, .free_count, .malloc_count, .total_gc_runs from within the github.com/armon/go-metrics library

• consul.serf.member.update and consul.serf.queue.* from within the github.com/hashicorp/serf library

[Amier3]

Hey @maxb

Thanks for bringing this to our attention, hopefully we'll get these in the docs shortly.

[deepto98]

Can I pick this up?

[nfi-hashicorp]

Hi @deepto98, sorry for the late reply! Are you still interested?

[deepto98]

Yeah, sure, Can I work on this?

[nfi-hashicorp]

Yeah, sure, Can I work on this?

Yes please and thank you!

[pglushko]

@deepto98 are you still working on this?