Skip to content

Commit 3d98d5d

Browse files
author
Jonathan S. Katz
committed
Make postgresVersion Helm variable required
This avoids some messiness that could occur on upgrades if this value is just a default, as one must explicitly set the value.
1 parent 850e369 commit 3d98d5d

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

helm/postgres/templates/postgres.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ kind: PostgresCluster
33
metadata:
44
name: {{ default .Release.Name .Values.name }}
55
spec:
6-
postgresVersion: {{ default 13 .Values.postgresVersion }}
6+
postgresVersion: {{ required "You must set the version of Postgres to deploy." .Values.postgresVersion }}
77
{{- if .Values.postGISVersion }}
88
postGISVersion: {{ quote .Values.postGISVersion }}
99
{{- end }}

helm/postgres/values.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,8 @@
1414
# postgresVersion sets the version to deploy. This version number needs to be
1515
# available as one of the "RELATED_IMAGE_POSTGRES_..." images as part of the PGO
1616
# installation if you want to deploy the image without setting the "postgres"
17-
# image variable. This defaults to the below value.
18-
# postgresVersion: 13
17+
# image variable. This value is required.
18+
postgresVersion: 14
1919

2020
# postGISVersion if sets and coupled with a PostGIS enabled container, enables
2121
# PostGIS. This version number needs to be available as one of the

0 commit comments

Comments
 (0)