Skip to content
This repository was archived by the owner on Feb 27, 2024. It is now read-only.

Commit cff0d67

Browse files
Chris Mellardjenkins-x-bot
Chris Mellard
authored andcommitted
fix(pencil): Fixed Velero values for AKS boot
Ensured initContainers in Velero values are only outputted if a namespace is defined for Velero Added in required resourceGroup for Velero Azure Plugin
1 parent ff91dda commit cff0d67

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

systems/velero/values.tmpl.yaml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ velero:
4242
bucket: {{ .Requirements.storage.backup.url | removeScheme | quote }}
4343
config:
4444
region: {{ .Requirements.cluster.region | quote }}
45-
{{- else if eq .Requirements.cluster.provider "aks" }}
45+
{{- else if and (eq .Requirements.cluster.provider "aks") (hasKey .Requirements.velero "namespace") }}
4646
initContainers:
4747
- name: velero-plugin-for-azure
4848
image: velero/velero-plugin-for-microsoft-azure:v1.0.0
@@ -54,9 +54,10 @@ velero:
5454
provider: azure
5555
backupStorageLocation:
5656
name: azure
57-
bucket: {{ .Requirements.storage.backup.url | removeScheme | quote }}
57+
bucket: {{ .Requirements.velero.bucketName | removeScheme | quote }}
5858
config:
5959
storageAccount: {{ .Requirements.velero.serviceAccount | quote }}
60+
resourceGroup: {{ .Requirements.velero.resourceGroup | quote }}
6061
{{- else if eq .Requirements.cluster.provider "iks" }}
6162
initContainers:
6263
- name: velero-plugin-for-aws

0 commit comments

Comments
 (0)