What motivated this proposal?
We have been observing OOMKill container crashes on the jsc container and have observed that setting the GOMEMLIMIT environment variable reduces memory usage and crashes, as recommended by this issue created in nats.
However, we currently manually modify the spec of the deployment to do this. There should be a way to set this in the helm chart.
What is the proposed change?
Add an env key, similar to how the NATS helm chart does for adding environment variables to the nats server container.
https://artifacthub.io/packages/helm/nats/nats?modal=values&path=container.env
Who benefits from this change?
No response
What alternatives have you evaluated?
No response
What motivated this proposal?
We have been observing OOMKill container crashes on the jsc container and have observed that setting the
GOMEMLIMITenvironment variable reduces memory usage and crashes, as recommended by this issue created in nats.However, we currently manually modify the spec of the deployment to do this. There should be a way to set this in the helm chart.
What is the proposed change?
Add an
envkey, similar to how the NATS helm chart does for adding environment variables to the nats server container.https://artifacthub.io/packages/helm/nats/nats?modal=values&path=container.env
Who benefits from this change?
No response
What alternatives have you evaluated?
No response