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

ScheduledBackup "schedule" paremeter fails with some values #288

Closed
walterbaidal opened this issue May 23, 2024 · 1 comment
Closed

ScheduledBackup "schedule" paremeter fails with some values #288

walterbaidal opened this issue May 23, 2024 · 1 comment
Labels
chart( cluster ) Related to the cluster chart

Comments

@walterbaidal
Copy link
Contributor

Hi,

When trying this configuration because of high RPO requirements:

  scheduledBackups:
    - name: every-four-hours-backup
      schedule: "* */4 * * *"
      backupOwnerReference: self

I get this error when helm installing:

Error: UPGRADE FAILED: YAML parse error on postgres-cluster/charts/cluster/templates/scheduled-backups.yaml: error converting YAML to JSON: yaml: line 13: did not find expected alphabetic or numeric character

A workaround to fix this problem I have already experienced in the past is adding extra single quotes in the schedule parameter:

  scheduledBackups:
    - name: every-four-hours-backup
      schedule: '"* */4 * * *"'
      backupOwnerReference: self

But to fully solve this issue, a change in the template is required. I will open a PR after creating this issue.

walterbaidal added a commit to walterbaidal/charts that referenced this issue May 23, 2024
itay-grudev pushed a commit that referenced this issue May 24, 2024
Fixes Issue #288 - schedule parameter not quoted

Signed-off-by: walterbaidal <[email protected]>
@itay-grudev
Copy link
Collaborator

Closed as completed. I will try to merge several other things into main and release them together.

@itay-grudev itay-grudev added the chart( cluster ) Related to the cluster chart label May 25, 2024
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

No branches or pull requests

2 participants