-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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 setting for filtering empty containers on Prometheus queries #7777
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: Juho Heikka <juho.heikka@gmail.com>
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
Can this PR get merged? |
Any progress? Would be great to get it into the next release. |
@jweak please resolve the conflicts, community really need this |
@jim-docker @ixrock please review this PR |
Hi! There is any update on this please? |
It appears that this repo is no longer maintained. |
very sad to see that this project is dead |
Adds a setting for applying the old Prometheus queries which were changed in this PR #6679
This change made it so that some setups are observing metrics as doubled.
container!=""
does filter those values.We have this change only for operator provider since it's the only one that had that change. We'll show the setting for Auto Detect or Operator values since those are the ones that could lead into using the operator Prometheus provider.
#7299