-
Notifications
You must be signed in to change notification settings - Fork 50
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
Replace Prometheus with Mimir #49
Conversation
This is a bit of a hack in the interests of making this change as small as possible - jsonnet-libs/prometheus-ksonnet imports its own implementation of a Grafana setup rather than using (say) jsonnet-libs/grafana.
e58a4d8
to
351d3f4
Compare
351d3f4
to
0c67088
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.
I haven't tried it out, but it looks pretty good to me!
My only concern is not configuring the location of the filesystem storage.
We could also consider if we want to use real object storage, but I don't think it is necessary (and would involve us deploying something like minio).
5d70ae6
to
09f9254
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.
Thanks for that. LGTM :-)
This PR replaces the Prometheus instance with a Mimir instance instead.
I did this as an exercise for myself, but thought it might be interesting to share. Will not be offended if this is rejected as an undesirable change :)
Outstanding work
Improvements for future PRs
jsonnet-libs/prometheus-ksonnet
rather than switching to something likejsonnet-libs/grafana
in the interest of keeping the PR small.