Helm-charts: Dependent charts of prometheus-operator?

[sachinmsft]

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. ?

[sachinmsft]

@scottrigby

[scottrigby]

@sachinmsft This was asked in a long thread of collapsed comments on #14. Here was my reply: https://github.com/scottrigby/prometheus-helm-charts/pull/1#discussion_r474766376

Do you have any other thoughts about this?

[monotek]

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

[scottrigby]

Re grafana I thought I replied to them last week but can't find the issue. So just made this comment: https://github.com/grafana/grafana/issues/18665#issuecomment-683425843

[scottrigby]

Also see https://github.com/helm/charts/pull/23662

[scottrigby]

Update: stable/grafana has officially deprecated and moved to a new repo 🙌 So we should update that dependency definition.

[gkarthiks]

@scottrigby should we change the dependency after the rename of prom-operator?

[scottrigby]

@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

[gkarthiks]

@scottrigby No worries, I can update this soon after the #1 is done.

[scottrigby]

If you want you could make it sooner against the #1 PR branch instead of main

[gkarthiks]

@scottrigby is there anything else we have to do against this issue here?

[scottrigby]

I don't think so. We can follow up with kube-state-metrics here https://github.com/kubernetes/kube-state-metrics/issues/1153 but until then, nothing is needed. Thanks!