Merge pull request #107 from lamvng/main

feat(backup-restore): Add flags to allow disable backup/restore hook …

Author: JC5

charts/firefly-db/README.md

@@ -37,6 +37,7 @@ storage:
 | backup.pvc.dataSize | string | `"1Gi"` |  |
 | backup.pvc.existingClaim | string | `""` | Use an existing PersistentVolumeClaim, overrides values above |
 | backupSchedule | string | `"0 3 * * *"` |  |
+| configs.BACKUP_ENABLED | bool | `true` | Enable backup hook job |
 | configs.BACKUP_URL | string | `""` |  |
 | configs.DBHOST | string | `""` |  |
 | configs.DBNAME | string | `"firefly"` |  |
@@ -46,6 +47,7 @@ storage:
 | configs.POSTGRES_HOST_AUTH_METHOD | string | `"trust"` |  |
 | configs.POSTGRES_PASSWORD | string | `""` |  |
 | configs.POSTGRES_USER | string | `"firefly"` |  |
+| configs.RESTORE_ENABLED | bool | `true` | Enable restore hook job |
 | configs.RESTORE_URL | string | `""` |  |
 | configs.TZ | string | `"Europe/Amsterdam"` |  |
 | configs.existingSecret | string | `""` | Set this to the name of a secret to load environment variables from. If defined, values in the secret will override values in configs |

charts/firefly-db/templates/firefly-db-backup-job.yml

@@ -1,3 +1,4 @@
+{{- if .Values.configs.BACKUP_ENABLED }}
 {{- if and (eq .Values.backup.destination "http") (empty .Values.configs.BACKUP_URL) }}
 {{- fail "BACKUP_URL must be set when backup.destination is 'http'" }}
 {{- end }}
@@ -23,3 +24,4 @@ spec:
   completions: 1
   template:
     {{- include "backupJobSpec" . | indent 4 }}
+{{- end }}

charts/firefly-db/templates/firefly-db-restore-job.yml

@@ -1,3 +1,4 @@
+{{- if .Values.configs.RESTORE_ENABLED }}
 apiVersion: batch/v1
 kind: Job
 metadata:
@@ -57,3 +58,4 @@ spec:
           {{- else }}
           emptyDir: {}
           {{- end }}
+{{- end }}

charts/firefly-db/values.yaml

@@ -21,6 +21,8 @@ backup:
     existingClaim: ""
 
 configs:
+  RESTORE_ENABLED: true
+  BACKUP_ENABLED: true
   RESTORE_URL: ""
   BACKUP_URL: ""  # -- Must have value if backup.destination is set to http
   PGPASSWORD: ""