Skip to content

feat(cluster): expose backups.target option in helm values.#847

Open
cloudymax wants to merge 1 commit into
cloudnative-pg:mainfrom
cloudymax:feat/backup-target
Open

feat(cluster): expose backups.target option in helm values.#847
cloudymax wants to merge 1 commit into
cloudnative-pg:mainfrom
cloudymax:feat/backup-target

Conversation

@cloudymax

Copy link
Copy Markdown

Closes #846

Exposes the backup.target option in the helm values.yaml file.
This will allow users to specify if they would like on-demand-backups to pull from the primary or a standby replica.

Supports 2 options:

  • primary
  • prefer-standby (default)

@cloudymax
cloudymax requested a review from itay-grudev as a code owner April 4, 2026 16:57
@dosubot dosubot Bot added size:S This PR changes 10-29 lines, ignoring generated files. chart( cluster ) Related to the cluster chart labels Apr 4, 2026
Signed-off-by: cloudymax <admin@cloudydev.net>
@cloudymax
cloudymax force-pushed the feat/backup-target branch from 9e4d16c to 282456a Compare April 4, 2026 17:04
@cloudymax cloudymax changed the title [FEAT] Expose backups.target option in helm values. feat(cluster): expose backups.target option in helm values. Apr 4, 2026
@jessebot

jessebot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

@itay-grudev could you please take a look and see if this makes sense for @cloudymax to fix the conflicts on? I think I'm still running into the issue described in #846

@itay-grudev

Copy link
Copy Markdown
Member

Yes. I like this. If you could resolve the conflicts and add the parameter in the non default configuration test I'll merge it.
https://github.com/cloudnative-pg/charts/tree/main/charts/cluster/test/postgresql-cluster-configuration

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 size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature Request] Expose backups.target option so users may choose to backup from the primary instance

3 participants