Added TTL to all Metrics exported

gluster / gluster-prometheus

Gluster monitoring using Prometheus

GNU Lesser General Public License v2.1

119 stars 70 forks source link

Added TTL to all Metrics exported #159

Closed aravindavk closed 5 years ago

aravindavk commented 5 years ago

Reseting all metrics causes missing entries since the Prometheus server may pull the metrics from exporter when it was reset. To avoid the problem, TTL option is provided(Default is 2 minutes).

If a Metric is not exported for the TTL period, then it deletes that metric from exporter to avoid stale entries.

Fixes: #158 Signed-off-by: Aravinda VK avishwan@redhat.com

aravindavk commented 5 years ago

WIP. Testing is pending and support for using configured value is pending.

sidharthanup commented 5 years ago

LGTM

aravindavk commented 5 years ago

CI is broken, I will send separate PR to use golangci-lint instead of gometalinter(gometalinter is deprecated)

@shtripat @sidharthanup Thanks for the review.