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
Control plane metrics #561
Conversation
I skip etcd in this PR because I'm not really sure which metrics do we expect. We already send some |
I recommend unified view for this PR (because of helm templates 😢 ) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, I love the templating for output section, personally I did not know about this functionality of helm, thanks for sharing 😄
c3080f2
to
4a4b12f
Compare
Fix according to misunderstanding of writeRelabelConfigs prometheus/prometheus#3996
We can limit scope using additional conditions if the list of exact names will be known
b4644ef
to
a066e0d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 LGTM
Description
Add control plane metrics:
Also extract match section from the metrics configuration to the helm template function
Testing performed