fix(base-cluster/backup): add defaultBackupStorageLocation flag for v…

…elero 🙄 (#779)
teutonet · Feb 12, 2024 · d0e9ff9 · d0e9ff9
1 parent d0bf855
commit d0e9ff9
Showing 1 changed file with 1 addition and 6 deletions.
diff --git a/charts/base-cluster/templates/backup/velero.yaml b/charts/base-cluster/templates/backup/velero.yaml
@@ -55,7 +55,7 @@ spec:
       resources: {{- .Values.backup.nodeAgent.resources | toYaml | nindent 8 }}
       priorityClassName: system-cluster-critical
     configuration:
-      {{- if .Values.backup.backupStorageLocations }}
+      defaultBackupStorageLocation: {{ .Values.backup.defaultLocation | quote }}
       backupStorageLocation: {{- range $name, $spec := .Values.backup.backupStorageLocations }}
         {{- $providerName := include "base-cluster.backup.getProviderName" $spec.provider }}
         - name: {{ $name | quote }}
@@ -64,9 +64,7 @@ spec:
           {{- if $spec.prefix }}
           prefix: {{ $spec.prefix | quote }}
           {{- end }}
-          {{- if $.Values.backup.defaultLocation }}
           default: {{ eq $name $.Values.backup.defaultLocation }}
-          {{- end }}
           {{- $credentialType := include "base-cluster.backup.credentialType" $spec.provider }}
           {{- if eq $credentialType "direct" }}
           credential:
@@ -86,9 +84,6 @@ spec:
             s3Url: {{ $providerSpec.url | quote }}
           {{- end }}
       {{- end }}
-      {{- else }}
-      backupStorageLocation: []
-      {{- end }}
       volumeSnapshotLocation: []
     metrics:
       serviceMonitor: