mimir icon indicating copy to clipboard operation
mimir copied to clipboard

helm: promote autoscaling to stable

Open dimitarvdimitrov opened this issue 1 year ago • 6 comments

https://github.com/grafana/mimir/pull/7282 added autoscaling to Helm as an experimental feature. This issue is about remaining work in order to promote autoscaling to stable.

Bugs

  • [x] https://github.com/grafana/mimir/issues/7473

Docs/Migration procedure

  • [ ] https://github.com/grafana/mimir/issues/7367

Remote URL UX

  • [x] Support a remote different from the metamonitoring setup https://github.com/grafana/mimir/pull/7392
  • [ ] Take the additional headers and auth from metamonitoring setup. Currently basic auth and extra headers are ignored.
  • [ ] Default to X-Scope-OrgID: metamonitoring if the config is already sending metrics to the same Mimir installation (the same way that metamonitoring computes it)
  • [ ] Add validation that is same mimir cluster is used, then metamonitoring is also enabled

Helm-jsonnet diffing

  • [ ] Add helm-jsonnet diffing so that the autoscaling configs don't get out of sync when we change one and forget to change the other. This is a matter of enabling autoscaling on select components in these two files and then making sure there are no differences between the rendered manifests. Minor differences can still be ignored via kustomiztions like this one

Make dashbaords compatible with helm-deployed KEDA objects

  • [ ] Similar to the /operations/mixin changes in https://github.com/grafana/mimir/pull/4687 - the matchers for keda objects should allow for helm-deployed objects

dimitarvdimitrov avatar Feb 12 '24 19:02 dimitarvdimitrov

@dimitarvdimitrov if you want, you can assign me to both of these issues. #7368 & #7367

beatkind avatar Feb 13 '24 09:02 beatkind

thanks @beatkind 🙏 I'm not sure of the level of detail in these issues, so ask away if anything isn't 100% clear

dimitarvdimitrov avatar Feb 13 '24 09:02 dimitarvdimitrov

Some thoughts on

Support a remote different from the metamonitoring setup

https://github.com/grafana/mimir/pull/7282#discussion_r1487522360

Basically drills down to have a global section for kedaAutoscaling:

kedaAutoscaling:
     prometheusAddress: http://... 
     customHeaders:
       {}
     pollingInterval: 10

beatkind avatar Feb 13 '24 10:02 beatkind

@dimitarvdimitrov is there an issue to add hpa to the components that do not have it yet like the ingester?

QuentinBisson avatar Apr 04 '24 13:04 QuentinBisson

for ingesters I could only find an internal one unfortunately - grafana/mimir-squad#1410. I see that @jhalterman was last working on that. Jonathan is there a public issue for this work?

dimitarvdimitrov avatar Apr 08 '24 09:04 dimitarvdimitrov

The issue you cited is the only one. There's nothing public yet.

jhalterman avatar Apr 08 '24 14:04 jhalterman