From 88e04bc4699e1335b7ff56fb402fbb5699487ce6 Mon Sep 17 00:00:00 2001
From: Fabio <fabiorauber@gmail.com>
Date: Wed, 10 May 2023 10:17:29 -0300
Subject: [PATCH] Update SAPL backup to use snapshots on weekdays

---
 charts/sapl/v0.2.0/templates/velero-schedule-monthly.yaml | 8 ++++----
 .../sapl/v0.2.0/templates/velero-schedule-weekdays.yaml   | 4 ++--
 charts/sapl/v0.2.0/templates/velero-schedule-weekly.yaml  | 4 ++--
 charts/sapl/v0.2.0/values.yaml                            | 2 --
 4 files changed, 8 insertions(+), 10 deletions(-)

diff --git a/charts/sapl/v0.2.0/templates/velero-schedule-monthly.yaml b/charts/sapl/v0.2.0/templates/velero-schedule-monthly.yaml
index 1d1f042..9f62eb8 100644
--- a/charts/sapl/v0.2.0/templates/velero-schedule-monthly.yaml
+++ b/charts/sapl/v0.2.0/templates/velero-schedule-monthly.yaml
@@ -14,10 +14,10 @@ spec:
     excludedResources:
       {{- toYaml . | nindent 8 }}
     {{- end }}
-    snapshotVolumes: {{ .Values.velero.backup.snapshotVolumes }}
-    # every weekday backup is good for the next year (365 days)
-    ttl: 8760h0m0s
-    defaultVolumesToRestic: {{ .Values.velero.backup.defaultVolumesToRestic }}
+    snapshotVolumes: false
+    # every monthly backup is good for the 6 months (~180 days)
+    ttl: 4380h0m0s
+    defaultVolumesToFsBackup: true
     hooks:
       resources:
       - name: pgdump
diff --git a/charts/sapl/v0.2.0/templates/velero-schedule-weekdays.yaml b/charts/sapl/v0.2.0/templates/velero-schedule-weekdays.yaml
index f0724d5..6d71bf8 100644
--- a/charts/sapl/v0.2.0/templates/velero-schedule-weekdays.yaml
+++ b/charts/sapl/v0.2.0/templates/velero-schedule-weekdays.yaml
@@ -14,10 +14,10 @@ spec:
     excludedResources:
       {{- toYaml . | nindent 8 }}
     {{- end }}
-    snapshotVolumes: {{ .Values.velero.backup.snapshotVolumes }}
+    snapshotVolumes: true
     # every weekday backup is good for the next week (7 days)
     ttl: 168h0m0s
-    defaultVolumesToRestic: {{ .Values.velero.backup.defaultVolumesToRestic }}
+    defaultVolumesToFsBackup: false
     hooks:
       resources:
       - name: pgdump
diff --git a/charts/sapl/v0.2.0/templates/velero-schedule-weekly.yaml b/charts/sapl/v0.2.0/templates/velero-schedule-weekly.yaml
index 9c07521..40ea067 100644
--- a/charts/sapl/v0.2.0/templates/velero-schedule-weekly.yaml
+++ b/charts/sapl/v0.2.0/templates/velero-schedule-weekly.yaml
@@ -14,10 +14,10 @@ spec:
     excludedResources:
       {{- toYaml . | nindent 8 }}
     {{- end }}
-    snapshotVolumes: {{ .Values.velero.backup.snapshotVolumes }}
+    snapshotVolumes: false
     # every sunday backup is good for 30 days (aprox. 1 month)
     ttl: 744h0m0s
-    defaultVolumesToRestic: {{ .Values.velero.backup.defaultVolumesToRestic }}
+    defaultVolumesToFsBackup: true
     hooks:
       resources:
       - name: pgdump
diff --git a/charts/sapl/v0.2.0/values.yaml b/charts/sapl/v0.2.0/values.yaml
index 342e43d..64fd9b0 100644
--- a/charts/sapl/v0.2.0/values.yaml
+++ b/charts/sapl/v0.2.0/values.yaml
@@ -78,8 +78,6 @@ velero:
   namespace: velero
   backup:
     enabled: true
-    snapshotVolumes: false
-    defaultVolumesToRestic: true
     # cert-manager objects are usually blocked during backup
     excludedResources: 
       - certificates.cert-manager.io