Skip to content

fix(cluster): critical reliability and templating fixes#827

Open
philippemnoel wants to merge 1 commit intocloudnative-pg:mainfrom
paradedb:phil/critical-reliability-fixes
Open

fix(cluster): critical reliability and templating fixes#827
philippemnoel wants to merge 1 commit intocloudnative-pg:mainfrom
paradedb:phil/critical-reliability-fixes

Conversation

@philippemnoel
Copy link
Copy Markdown
Contributor

I caught these while working on the @paradedb fork. Figured these changes would benefit upstream and reduce our diff:

  • Fix PID comparison in logical replication stopped alerts: == "" doesn't work for numeric metrics, changed to <= 0
  • Fix endpointCA in barman object store template: use coalesce/default so it falls back to the generated CA bundle name and default key when only .endpointCA.create is set without explicit .name/.key
  • Fix databases.yaml templating:
    • Default database owner to the database name when .owner is not specified
    • Use hasKey instead of with for isTemplate, allowConnections, and connectionLimit so that false/0 values are correctly rendered instead of silently skipped
    • Default schema owner to the database owner when not explicitly set

Enjoy!

@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. bug Something isn't working chart( cluster ) Related to the cluster chart labels Mar 5, 2026
Signed-off-by: Philippe Noël <philippemnoel@gmail.com>
@philippemnoel philippemnoel force-pushed the phil/critical-reliability-fixes branch from d35d736 to 0ccb248 Compare March 5, 2026 20:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working chart( cluster ) Related to the cluster chart size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant