Skip to content

Commit 72648ac

Browse files
committed
fix(helm): revert copilot-postgresql STS serviceName too (same immutability issue)
Audit caught that the main fix in d5c2e8e missed statefulset-copilot-postgres.yaml, which had the identical immutable-field rename from -copilot-postgresql to -copilot-postgresql-headless. Same upgrade-break vector for anyone running copilot.enabled=true on a prior chart version. Mirrors the fix and comment from the main postgresql STS.
1 parent e122858 commit 72648ac

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

helm/sim/templates/statefulset-copilot-postgres.yaml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,11 @@ metadata:
6666
{{- include "sim.labels" . | nindent 4 }}
6767
app.kubernetes.io/component: copilot-postgresql
6868
spec:
69-
serviceName: {{ include "sim.fullname" . }}-copilot-postgresql-headless
69+
# Must remain {{ include "sim.fullname" . }}-copilot-postgresql (not the
70+
# -headless name) — spec.serviceName is immutable on a StatefulSet, and
71+
# the prior chart shipped with this value. Same rationale as the main
72+
# postgresql STS; see statefulset-postgresql.yaml for details.
73+
serviceName: {{ include "sim.fullname" . }}-copilot-postgresql
7074
replicas: 1
7175
podManagementPolicy: OrderedReady
7276
updateStrategy:

0 commit comments

Comments
 (0)