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.

Sign up for GitHub

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

support: recovery target variables in helm chart #399

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

support: recovery target variables in helm chart #399

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
622a311
support: backupID in helm chart
sadath-12 Sep 24, 2024
2c572b7
fix: correct backupID field
sadath-12 Sep 24, 2024
b5fd51e
re-order and add other supported target recoveries
sadath-12 Sep 24, 2024
a409141
add missing target params
sadath-12 Sep 25, 2024
e18764f
Merge branch 'main' into backup
sadath-12 Sep 25, 2024
5b20a25
Merge branch 'main' into backup
sadath-12 Sep 30, 2024
dcf6f20
Merge branch 'main' into backup
itay-grudev Oct 2, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
21 changes: 20 additions & 1 deletion charts/cluster/templates/_bootstrap.tpl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -71,10 +71,29 @@ externalClusters:

{{- else }}
recovery:
{{- with .Values.recovery.pitrTarget.time }}

{{- if or .Values.recovery.pitrTarget.targetTime .Values.recovery.pitrTarget.backupID .Values.recovery.pitrTarget.targetXID .Values.recovery.pitrTarget.targetName .Values.recovery.pitrTarget.targetLSN .Values.recovery.pitrTarget.targetImmediate }}
recoveryTarget:
{{- with .Values.recovery.pitrTarget.targetTime }}
targetTime: {{ . }}
{{- end }}
{{- with .Values.recovery.pitrTarget.backupID }}
backupID: {{ . }}
{{- end }}
{{- with .Values.recovery.pitrTarget.targetXID }}
targetXID: {{ . }}
{{- end }}
{{- with .Values.recovery.pitrTarget.targetName }}
targetName: {{ . }}
{{- end }}
{{- with .Values.recovery.pitrTarget.targetLSN }}
targetLSN: {{ . }}
{{- end }}
{{- with .Values.recovery.pitrTarget.targetImmediate }}
targetImmediate: {{ . }}
{{- end }}
{{- end }}

{{- if eq .Values.recovery.method "backup" }}
backup:
name: {{ .Values.recovery.backupName }}
Expand Down
12 changes: 11 additions & 1 deletion charts/cluster/values.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,17 @@
## -- Point in time recovery target. Specify one of the following:
pitrTarget:
# -- Time in RFC3339 format
time: ""
targetTime: ""
# -- immediate target given a backup id
backupID: ""
# -- Transaction ID up to which recovery proceeds. (The precise stopping point is also influenced by the exclusive option.)

Check failure on line 43 in charts/cluster/values.yaml

View workflow job for this annotation

GitHub Actions / linter 

43:128 [trailing-spaces] trailing spaces
targetXID: ""
# -- Named restore point (created with pg_create_restore_point()) to which recovery proceeds
targetName: ""
# -- LSN of the write-ahead log location up to which recovery proceeds. (The precise stopping point is also influenced by the exclusive option.)
targetLSN: ""
# -- Recovery ends as soon as a consistent state is reached, that is, as early as possible
targetImmediate: ""

##
# -- Backup Recovery Method
Expand Down
Loading