fix copilot remarks

Author: LiseMachetel

charts/linkurious-enterprise/README.md

@@ -65,23 +65,24 @@ helm upgrade --install my-release linkurious-enterprise-0.3.1.tgz -f chart-value
 | backup.litestream.resources.requests.cpu | string | `"100m"` |  |
 | backup.litestream.resources.requests.memory | string | `"100Mi"` |  |
 | backup.litestream.secretRef.name | string | `"litestream-lke-secret"` |  |
+| backup.velero.enabled | bool | `false` |  |
 | backup.velero.includedResources[0] | string | `"pvc"` |  |
 | backup.velero.includedResources[1] | string | `"pv"` |  |
 | backup.velero.instance | string | `"velero-preprod"` |  |
 | backup.velero.labels."app.kubernetes.io/name" | string | `"linkurious-enterprise"` |  |
 | backup.velero.labelsSelector."app.kubernetes.io/name" | string | `"linkurious-enterprise"` |  |
 | backup.velero.name | string | `"velero"` |  |
 | backup.velero.primaryBackup.defaultVolumesToFsBackup | bool | `false` |  |
-| backup.velero.primaryBackup.enabled | bool | `false` |  |
 | backup.velero.primaryBackup.schedule | string | `"55 3 * * *"` |  |
+| backup.velero.primaryBackup.snapshotMoveData | bool | `false` |  |
 | backup.velero.primaryBackup.snapshotVolume | bool | `true` |  |
 | backup.velero.primaryBackup.ttl | string | `"168h"` |  |
 | backup.velero.secondaryBackup.defaultVolumesToFsBackup | bool | `true` |  |
 | backup.velero.secondaryBackup.enabled | bool | `false` |  |
 | backup.velero.secondaryBackup.schedule | string | `"55 12 * * *"` |  |
 | backup.velero.secondaryBackup.snapshotMoveData | bool | `true` |  |
 | backup.velero.secondaryBackup.snapshotVolume | bool | `true` |  |
-| backup.velero.secondaryBackup.storageLocation | string | `"nl-ams-backup-storage"` |  |
+| backup.velero.secondaryBackup.storageLocation | string | `"velero-k8s-nl-ams-backup-storage"` |  |
 | backup.velero.secondaryBackup.ttl | string | `"168h"` |  |
 | backup.velero.skipImmediately | bool | `false` |  |
 | backup.velero.veleroNamespace | string | `"backup"` |  |

charts/linkurious-enterprise/templates/statefulset.yaml

@@ -19,7 +19,7 @@ spec:
       {{- with .Values.podAnnotations }}
         {{- toYaml . | nindent 8 }}
       {{- end }}
-      {{- if .Values.backup.velero.primaryBackup.enabled }}
+      {{- if or (eq .Values.backup.velero.backup.enabled true) (eq .Values.backup.velero.secondaryBackup.enabled true) }}
         backup.velero.io/backup-volumes: lke-data
       {{- end }}
       labels:

charts/linkurious-enterprise/templates/velero-schedule.yaml

@@ -1,4 +1,4 @@
-{{- if .Values.backup.velero.primaryBackup.enabled }}
+{{- if .Values.backup.velero.Backup.enabled }}
 {{- if (not (.Capabilities.APIVersions.Has "velero.io/v1")) }}
     {{- fail "ERROR: You have to deploy velero.io/v1 first or disable velero backup" }}
 {{- end }}

charts/linkurious-enterprise/values.yaml

@@ -281,6 +281,7 @@ backup:
 
   velero:
     ## -- Enable backup for Linkurious server lke-data volume with velero
+    enabled: false
     name: velero
     instance: velero-preprod
     veleroNamespace: backup
@@ -296,7 +297,6 @@ backup:
       # backup.velero.io/backup-volumes: lke-data
       app.kubernetes.io/name: linkurious-enterprise
     primaryBackup:
-      enabled: false
       snapshotVolume: true
       snapshotMoveData: false
       defaultVolumesToFsBackup: false