Skip to content

Commit

Permalink
Add walStorage capability and remove gkeEnvironment conflict in clust…
Browse files Browse the repository at this point in the history
…er chart. (#312)

* Add walStorage option to cluster.yaml template
* Bug Fix: Remove applicationCredentials when gktEnvironment is true.

Signed-off-by: Jon <[email protected]>
  • Loading branch information
rocket357 authored Jun 15, 2024
1 parent 50666bd commit fd5eff9
Show file tree
Hide file tree
Showing 4 changed files with 22 additions and 1 deletion.
2 changes: 2 additions & 0 deletions charts/cluster/templates/_barman_object_store.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -62,8 +62,10 @@
{{- $secretName := coalesce .scope.secret.name (printf "%s-%s-google-creds" .chartFullname .secretPrefix) }}
googleCredentials:
gkeEnvironment: {{ .scope.google.gkeEnvironment }}
{{- if not .scope.google.gkeEnvironment }}
applicationCredentials:
name: {{ $secretName }}
key: APPLICATION_CREDENTIALS
{{- end }}
{{- end -}}
{{- end -}}
6 changes: 5 additions & 1 deletion charts/cluster/templates/cluster.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,11 @@ spec:
storage:
size: {{ .Values.cluster.storage.size }}
storageClass: {{ .Values.cluster.storage.storageClass }}

{{- if .Values.cluster.walStorage }}

This comment has been minimized.

Copy link
@Stevenpc3

Stevenpc3 Aug 9, 2024

Contributor

Many things in this MR should have bumped the chart.yaml version for cluster

walStorage:
size: {{ .Values.cluster.walStorage.size }}
storageClass: {{ .Values.cluster.walStorage.storageClass }}
{{- end }}
{{- with .Values.cluster.resources }}
resources:
{{- toYaml . | nindent 4 }}
Expand Down
11 changes: 11 additions & 0 deletions charts/cluster/values.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -270,6 +270,17 @@
}
}
},
"walStorage": {
"type": "object",
"properties": {
"size": {
"type": "string"
},
"storageClass": {
"type": "string"
}
}
},
"superuserSecret": {
"type": "string"
}
Expand Down
4 changes: 4 additions & 0 deletions charts/cluster/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -101,6 +101,10 @@ cluster:
storage:
size: 8Gi
storageClass: ""

walStorage:
size: 1Gi
storageClass: ""

# -- The UID of the postgres user inside the image, defaults to 26
postgresUID: 26
Expand Down

0 comments on commit fd5eff9

Please sign in to comment.