Skip to content

Commit 756cb00

Browse files
IvanHuntersIvanHunters
committed
fix(postgres): correct merge argument order for parameter precedence
Fix regression where defaults were overriding user-supplied parameters. Sprig merge gives precedence to first argument, so user values must be merged before defaults to maintain correct override behavior. Before: merge (dict) $defaults .Values.postgresql.parameters After: merge (dict) .Values.postgresql.parameters $defaults Now max_wal_senders and other user parameters correctly override defaults. Signed-off-by: IvanHunters <[email protected]>
1 parent 3b049b7 commit 756cb00

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

  • packages/apps/postgres/templates

‎packages/apps/postgres/templates/db.yaml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ spec:
5858
postgresql:
5959
parameters:
6060
{{- $defaults := dict "max_wal_senders" "30" }}
61-
{{- $params := merge (dict) $defaults .Values.postgresql.parameters }}
61+
{{- $params := merge (dict) .Values.postgresql.parameters $defaults }}
6262
{{- /* Security: Block parameters that could allow shell command execution */ -}}
6363
{{- $dangerousParams := list "archive_command" "restore_command" "ssl_passphrase_command" "log_destination" "event_source" }}
6464
{{- range $key, $value := $params }}

0 commit comments

Comments
 (0)