Add a way to configure node.js heap size.

vprus commented 4 months ago

The Cube's production checklist (https://cube.dev/docs/product/deployment/production-checklist#memory-and-cpu) recommends that refresh worker has 6GB of memory, and also say that NODE_OPTIONS must be adjusted for it to use that memory.

The only way to accomplish that with the current chart is the extraEnvVars variable. However, it also applies to the API pods, so it's not possible to give different memory to API and worker.

It would be great if either:

there is 'memory' option for both API and workers, which is translated into both K8S resources and into NODE_OPTIONS env var.
or, there are separate env var setting for API and worker.

lvauvillier commented 3 months ago

Hi @vprus, thanks a lot for reporting this. I'm currently working on a new release, i'll add the env var setting option.

lvauvillier commented 3 months ago

Fixed.

Just add this config to your values.yaml file:

worker:
  extraEnvVars:
    - name: NODE_OPTIONS
      value: --max-old-space-size=6144

gadsme / charts

Add a way to configure node.js heap size. #45