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

Add Grafana Datasource support and Dashboards #34

Merged
merged 9 commits into from
Feb 19, 2024

Conversation

lucabello
Copy link
Contributor

@lucabello lucabello commented Feb 2, 2024

This PR does two things:

  • add support for Mimir to be a Grafana Datasource
  • fix the dashboards we had in the coordinator so they actually work

This contributes to #16, but doesn't close it yet.


About the dashboards, there's not really a good way for you to validate my changes. Some panels are also still gonna be empty, because Mimir doesn't always generate metrics when the feature they relate to is not used.

I've made the dashboards the best we can have at the moment; testing with partner cloud will give us an opportunity to figure out what exactly works and what doesn't, and it could give us enough insight to make more useful dashboards oureselves :)


The other library changes you see are just charmcraft fetch-lib!

@lucabello
Copy link
Contributor Author

This PR needs canonical/grafana-k8s-operator#295 to be resolved.

@lucabello lucabello marked this pull request as draft February 2, 2024 15:56
@lucabello lucabello marked this pull request as ready for review February 16, 2024 10:48
@lucabello lucabello changed the title Add Grafana Datasource support Add Grafana Datasource support and Dashboards Feb 16, 2024
lib/charms/tempo_k8s/v1/charm_tracing.py Dismissed Show dismissed Hide dismissed
@lucabello lucabello merged commit 6022e46 into main Feb 19, 2024
13 checks passed
@lucabello lucabello deleted the feature/grafana-source branch February 19, 2024 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants