Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Re-order template keywords #389

Merged
merged 2 commits into from
Oct 9, 2024
Merged

Conversation

MoonMoon2
Copy link
Contributor

@MoonMoon2 MoonMoon2 commented Sep 16, 2024

Issue was because YAML output for this section of the cluster.yaml was

parameters:
parameters: {}
pg_hba: []
pg_indent: []

Whereas it should be

parameters: {}
pg_hba: {}
pg_indent: {}

but parameters is a helm keyword so it would consume all the keywords in the parameters step before pg_hba or pg_indent were called.

This simple re-ordering should fix the issue (and has for me in testing)

Closes: #408 #388

Issue was because YAML output for this section of the cluster.yaml was

parameters:
  parameters: {}
  pg_hba: []
  pg_indent: []

Whereas it should be

parameters: {}
pg_hba: {}
pg_indent: {}

but parameters is a helm keyword so it would consume all the keywords in the parameters step before pg_hba or pg_indent were called.

This simple re-ordering should fix the issue (and has for me in testing)

Signed-off-by: Tobias Ward <[email protected]>
@MoonMoon2 MoonMoon2 changed the title Fixed issue 388 Re-order template keywords Sep 16, 2024
@itay-grudev itay-grudev added the chart( cluster ) Related to the cluster chart label Oct 4, 2024
@itay-grudev itay-grudev merged commit d8f420b into cloudnative-pg:main Oct 9, 2024
4 checks passed
@hakro
Copy link

hakro commented Oct 9, 2024

Hi !
Any ETA for when the next release containing this fix will be published ? Just to know if I hack around the chart manually, or if I wait and then update it from upstream.

Thanx

@itay-grudev
Copy link
Collaborator

itay-grudev commented Oct 9, 2024

Wait for upstream. There's a bunch of people asking me for a release so I'll try to create one soon (this week).

P.S. Just be advised that it will contain some small breaking changes. I'll document everything in the release notes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chart( cluster ) Related to the cluster chart
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Chart cnpg/cluster duplicate key cluster.postgresql.parameters.
3 participants