As kube-state-metrics now has switched to including jsonnet we should remove ours in favour of that and bring it in as a dependency. Same goes for the alerts.
See https://github.com/coreos/kube-prometheus/pull/368#issuecomment-572694051
I can submit PR after holidays (beginning of Feb). would it be okay?
If no one will have done it by then, please! :)
Yes go for it! Thanks!
Hi, I had a quick look into the ksm lib on kube-prometheus, it contains additionally ksm autoscaler. Thus, there are 2 possibilities:
I think we should default to the standard deployment. We can figure out afterwards how people might be able to opt into the autosharding feature with the statefulset.
I would like to end up with the similar deployment we already have in this repo just that it uses the upstream jsonnet. In a follow up issue we should look at how to easily opt into the new autosharding feature. Main point of this issue is to remove the ned to manually update the ksm deployments.
I think this can be closed now. If we need to do follow up we can open a new issue.
Most helpful comment
I would like to end up with the similar deployment we already have in this repo just that it uses the upstream jsonnet. In a follow up issue we should look at how to easily opt into the new autosharding feature. Main point of this issue is to remove the ned to manually update the ksm deployments.