-
Notifications
You must be signed in to change notification settings - Fork 5.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
Dependent charts of prometheus-operator? #30
Comments
@sachinmsft This was asked in a long thread of collapsed comments on #14. Here was my reply: #1 (comment) Do you have any other thoughts about this? |
I guess grafana will be moved to an official grafana helm repo anyway. Loki for example is already managed by the grafana guys. See: https://hub.helm.sh/charts/loki/loki |
Re grafana I thought I replied to them last week but can't find the issue. So just made this comment: grafana/grafana#18665 (comment) |
Also see helm/charts#23662 |
Update: stable/grafana has officially deprecated and moved to a new repo 🙌 So we should update that dependency definition. |
@scottrigby should we change the dependency after the rename of prom-operator? |
@gkarthiks yes as a fast follow of #1 if we can get that tested and reviewed one more time and merged soon. Otherwise I can update as part of that PR |
@scottrigby No worries, I can update this soon after the #1 is done. |
If you want you could make it sooner against the #1 PR branch instead of main |
@scottrigby is there anything else we have to do against this issue here? |
I don't think so. We can follow up with kube-state-metrics here kubernetes/kube-state-metrics#1153 but until then, nothing is needed. Thanks! |
[prometheus-node-exporter] Integrate process-exporter and calico-exporter
I see that we are moving prometheus charts from stable to this repo. what about other charts that prometheus-operatror deploy like grafana, kube-state-metrics. ?
The text was updated successfully, but these errors were encountered: