-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Add metric successful_config_load #413
Conversation
Signed-off-by: Ryan Shatford <ryan.shatford@tapad.com>
470b763
to
72a36fd
Compare
I might also introduce a counter for counting all config load errors. |
I'm not sure that is useful, however Prometheus has a prometheus_config_last_reload_success_timestamp_seconds |
- Adds Namespace to prometheus metric - metric name more in line with prometheus standards - move metric Set() into a defer block Signed-off-by: Ryan Shatford <ryan.shatford@tapad.com>
I agree that the |
timestamp_seconds would have the value of the time, not a metric timestamp. |
Signed-off-by: Ryan Shatford <ryan.shatford@tapad.com>
0b4d7c5
to
d0958a3
Compare
ah, the timestamp of the last successful reload. I've added that metric and cleaned up the code a bit more |
Signed-off-by: Ryan Shatford <ryan.shatford@tapad.com>
yup, i just realized that. should be fixed now. |
Signed-off-by: Ryan Shatford <ryan.shatford@tapad.com>
Also move the init() func to under the var block Signed-off-by: Ryan Shatford <ryan.shatford@tapad.com>
e3bd402
to
f9923c4
Compare
Thanks! |
No description provided.