Skip to content

feat: Add extraEnvs to operator helm chart - #2671

Merged
FxKu merged 1 commit into
zalando:masterfrom
desaintmartin:master
Aug 6, 2024
Merged

FxKu merged 1 commit into
zalando:masterfrom
desaintmartin:master

Conversation

@desaintmartin

@desaintmartin desaintmartin commented Jun 25, 2024

Copy link
Copy Markdown
Contributor

Fixes #2672

Inspired from same feature from UI chart.

@FxKu

FxKu commented Jul 4, 2024

Copy link
Copy Markdown
Contributor

Can you remove the comments about UI?

Comment thread charts/postgres-operator/values.yaml Outdated
Comment thread charts/postgres-operator/values.yaml Outdated
Signed-off-by: Cédric de Saint Martin <cdesaintmartin@wiremind.io>
@FxKu

FxKu commented Jul 29, 2024

Copy link
Copy Markdown
Contributor

👍

1 similar comment
@idanovinda

Copy link
Copy Markdown
Collaborator

👍

@FxKu
FxKu merged commit ce15d10 into zalando:master Aug 6, 2024
@pat-trunk-io

Copy link
Copy Markdown

Shouldn't the indent be 10 not 12 to match the indentation of resources below?
{{- .Values.extraEnvs | toYaml | nindent 12 }}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

postgres-operator chart: Add extraEnvs support

4 participants