-
Notifications
You must be signed in to change notification settings - Fork 2.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
mixin: Remove assumed metrics #3854
Conversation
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.
Remove pod selectors
Why?
It's probably useful to continue to use the |
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.
Stil I might be stupid, but why we get rid of pod. Having some random IP address is not helpful on dashboards 🤔
The best practice is to relabel something sensible into your |
I just want to make our mixin kubernetes agnostic (this has been asked a couple of times) and I think these changes will do the work. |
Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com>
Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com>
Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com>
Add promdoc in toolchain Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com>
Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com>
I still see a bunch of occasions of the |
IMHO |
Discussed this offline and created an issue for it #3867 |
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.
Overall this looks really good! 👍
Happy to continue with namespace
in a follow up,
* Fix filenames Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com> * Remove assumed metric dependencies Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com> * Add changelog Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com> * Rename pod to instance Add promdoc in toolchain Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com> * Fix test Signed-off-by: Kemal Akkoyun <kakkoyun@gmail.com>
This PR fixes minor inconveniences with mixin. Remove assumed metrics. Use
thanos_info
instead ofkube_pod_info
for dashboard selectors.Changes
thanos_info
instead ofkube_pod_info
for dashboard selectors.Verification
make examples