From 8b95db06ee53508eb9afa23170191ebecd6b5d12 Mon Sep 17 00:00:00 2001 From: Timofei Larkin Date: Mon, 3 Nov 2025 12:37:56 +0300 Subject: [PATCH] [nats] Merge container spec, not podTemplate ## What this PR does The NATS chart incorrectly used podTemplate+merge instead of container+merge to add resource requests and limits to the NATS container in the statefulset, but as a result it just completely wiped out the default container spec. By moving the overrides under the container key, the upstream chart now correctly merges the resource requests, instead of overwriting the container spec. ### Release note ```release-note [nats] Fix incorrect path to container resources in the NATS chart. ``` Signed-off-by: Timofei Larkin --- packages/apps/nats/templates/nats.yaml | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/packages/apps/nats/templates/nats.yaml b/packages/apps/nats/templates/nats.yaml index 9994332f..55e0bd85 100644 --- a/packages/apps/nats/templates/nats.yaml +++ b/packages/apps/nats/templates/nats.yaml @@ -47,13 +47,9 @@ spec: retries: -1 values: nats: - podTemplate: + container: merge: - spec: - containers: - - name: nats - image: nats:2.10.17-alpine - resources: {{- include "cozy-lib.resources.defaultingSanitize" (list .Values.resourcesPreset .Values.resources $) | nindent 22 }} + resources: {{- include "cozy-lib.resources.defaultingSanitize" (list .Values.resourcesPreset .Values.resources $) | nindent 12 }} fullnameOverride: {{ .Release.Name }} config: {{- if or (gt (len $passwords) 0) (gt (len .Values.config.merge) 0) }}