diff --git a/README.adoc b/README.adoc index 4e47bec..0f37acb 100644 --- a/README.adoc +++ b/README.adoc @@ -216,11 +216,11 @@ The following providers are used by this module: - [[provider_random]] <> (>= 3) -- [[provider_utils]] <> (>= 1) +- [[provider_null]] <> - [[provider_argocd]] <> (>= 5) -- [[provider_null]] <> +- [[provider_utils]] <> (>= 1) === Resources @@ -299,7 +299,7 @@ Description: Override of target revision of the application chart. Type: `string` -Default: `"v3.4.0"` +Default: `"v3.7.0"` ==== [[input_helm_values]] <> @@ -588,8 +588,8 @@ Description: ID to pass other modules in order to refer to this module as a depe |=== |Name |Version |[[provider_random]] <> |>= 3 -|[[provider_utils]] <> |>= 1 |[[provider_argocd]] <> |>= 5 +|[[provider_utils]] <> |>= 1 |[[provider_null]] <> |n/a |=== @@ -656,7 +656,7 @@ Description: ID to pass other modules in order to refer to this module as a depe |[[input_target_revision]] <> |Override of target revision of the application chart. |`string` -|`"v3.4.0"` +|`"v3.7.0"` |no |[[input_helm_values]] <> diff --git a/charts/longhorn/templates/backup-storageclass.yaml b/charts/longhorn/templates/backup-storageclass.yaml index 2f71cce..8facb3e 100644 --- a/charts/longhorn/templates/backup-storageclass.yaml +++ b/charts/longhorn/templates/backup-storageclass.yaml @@ -22,26 +22,5 @@ parameters: {{- toYaml . | nindent 4 }} {{- end }} {{- end }} - recurringJobs: '[ - {{- if $.Values.backups.config.snapshot_enabled }} - { - "name":"snapshot", - "task":"snapshot", - "cron": "{{ $.Values.backups.config.snapshot_cron }}", - "retain": {{ $.Values.backups.config.snapshot_retention }} - } - {{- end -}} - {{- if and $.Values.backups.config.backup_enabled $.Values.backups.config.snapshot_enabled -}} - , - {{ end -}} - {{- if $.Values.backups.config.backup_enabled -}} - { - "name":"backup", - "task":"backup", - "cron": "{{ $.Values.backups.config.backup_cron }}", - "retain": {{ $.Values.backups.config.backup_retention }} - } - {{- end }} - ]' {{- end }} diff --git a/charts/longhorn/templates/recurring-job-backup.yaml b/charts/longhorn/templates/recurring-job-backup.yaml new file mode 100644 index 0000000..e5195ab --- /dev/null +++ b/charts/longhorn/templates/recurring-job-backup.yaml @@ -0,0 +1,20 @@ +{{- if $.Values.backups.enabled -}} +{{- if $.Values.backups.config.backup_enabled -}} + +--- +apiVersion: longhorn.io/v1beta2 +kind: RecurringJob +metadata: + name: {{ $.Release.Name }}-backup + labels: + name: {{ $.Release.Name }}-backup +spec: + concurrency: 1 + cron: {{ $.Values.backups.config.backup_cron | quote }} + labels: {} + name: backup + retain: {{ $.Values.backups.config.backup_retention }} + task: backup + +{{- end -}} +{{- end -}} diff --git a/charts/longhorn/templates/recurring-job-snapshot.yaml b/charts/longhorn/templates/recurring-job-snapshot.yaml new file mode 100644 index 0000000..21ee9dc --- /dev/null +++ b/charts/longhorn/templates/recurring-job-snapshot.yaml @@ -0,0 +1,20 @@ +{{- if $.Values.backups.enabled -}} +{{- if $.Values.backups.config.snapshot_enabled -}} + +--- +apiVersion: longhorn.io/v1beta2 +kind: RecurringJob +metadata: + name: {{ $.Release.Name }}-snapshot + labels: + name: {{ $.Release.Name }}-snapshot +spec: + concurrency: 1 + cron: {{ $.Values.backups.config.snapshot_cron | quote }} + labels: {} + name: backup + retain: {{ $.Values.backups.config.snapshot_retention }} + task: backup + +{{- end -}} +{{- end -}} diff --git a/locals.tf b/locals.tf index cb6093a..40f95f6 100644 --- a/locals.tf +++ b/locals.tf @@ -12,7 +12,6 @@ locals { defaultSettings = { backupTarget = var.enable_pv_backups ? format("s3://%s@%s/", var.backup_storage.bucket_name, var.backup_storage.region) : "" backupTargetCredentialSecret = var.enable_pv_backups ? "longhorn-s3-secret" : "" - defaultLonghornStaticStorageClass = var.enable_pv_backups && var.set_default_storage_class ? "longhorn-backup" : "longhorn-static" storageOverProvisioningPercentage = var.storage_over_provisioning_percentage storageMinimalAvailablePercentage = var.storage_minimal_available_percentage taintToleration = join(";", local.tolerations_list)