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

check if metric is registererd #4024

Closed
miekg opened this issue Jul 24, 2020 · 2 comments · Fixed by #4025
Closed

check if metric is registererd #4024

miekg opened this issue Jul 24, 2020 · 2 comments · Fixed by #4025

Comments

@miekg
Copy link
Member

miekg commented Jul 24, 2020

See #4021 We seem to be forgetting to register metrics (too often).

The exiting metrics naming test (test/metric_naming_test.go) should be extended to test for this as well - at least assuming it's easy enough to do so

@miekg
Copy link
Member Author

miekg commented Jul 24, 2020

See this thanos-io/thanos#2102
We need to fix our declaration and registration. I.e. the fix the code not the test

@chrisohaver
Copy link
Member

I.e. the fix the code not the test

I have opted to use promauto in some other (simpler) projects.

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.

2 participants