- Notifications
You must be signed in to change notification settings - Fork3.8k
Add query scheduler to "Read Resources" mixin dashboard#6028
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
This component has been around for a while but has not been added to theloki-mixin yet.Signed-off-by: Christian Haudum <christian.haudum@gmail.com>
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 👍.
Adding a screenshot or public preview link would be helpful to review PR(s) like this IMHO :)
Agree. Let me install the dashboards locally. |
@kavirajk Added a screenshot to the PR description. |
Can we stack CPU and memory graph or that is too much ? |
I don't see it yet. I see it on the other PR though :)https://github.com/grafana/deployment_tools/pull/30245. Feel free to go ahead on merge. It's a non blocker. |
@kavirajk 🤦 I'm pretty sure I posted it already. Here we go ... |
This component has been around for a while but has not been added to the
loki-mixin yet.
Signed-off-by: Christian Haudumchristian.haudum@gmail.com
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Checklist
CHANGELOG.md.docs/sources/upgrading/_index.md