Add resource limits

[thikoen]

Is it possible to add resource limits? Just found options for metrics and in the operator deployment itself but not for clickhouse within the values.
Should be possible as part of the pod template: https://github.com/Altinity/clickhouse-operator/blob/master/docs/chi-examples/02-templates-03-host-template-volume-claim-and-pod-resources-limit.yaml#L37
Maybe you can just add it also to the values in this chart.
Thanks

[joshleecreates]

We have resource requests and limits for Keeper, it would be good to also add them to ClickHouse. Thanks!

[3timeslazy]

@thikoen shouldn't that work?

clickhouse:
  resources:
    memory: ...

based on https://github.com/Altinity/helm-charts/blob/main/charts/clickhouse/templates/_helpers.tpl#L121-L122

[erwan-b]

This work fine.

clickhouse:
  resources:
    requests:
      cpu: 1000m
      memory: 2048Mi
    limits:
      memory: 2048Mi