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

Commit

Permalink
fix(pencil): Fixed Velero values for AKS boot
Browse files Browse the repository at this point in the history
Ensured initContainers in Velero values are only outputted if a namespace is defined for Velero

Added in required resourceGroup for Velero Azure Plugin
  • Loading branch information
Chris Mellard authored and jenkins-x-bot committed Jul 28, 2020
1 parent ff91dda commit cff0d67
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions systems/velero/values.tmpl.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ velero:
bucket: {{ .Requirements.storage.backup.url | removeScheme | quote }}
config:
region: {{ .Requirements.cluster.region | quote }}
{{- else if eq .Requirements.cluster.provider "aks" }}
{{- else if and (eq .Requirements.cluster.provider "aks") (hasKey .Requirements.velero "namespace") }}
initContainers:
- name: velero-plugin-for-azure
image: velero/velero-plugin-for-microsoft-azure:v1.0.0
Expand All @@ -54,9 +54,10 @@ velero:
provider: azure
backupStorageLocation:
name: azure
bucket: {{ .Requirements.storage.backup.url | removeScheme | quote }}
bucket: {{ .Requirements.velero.bucketName | removeScheme | quote }}
config:
storageAccount: {{ .Requirements.velero.serviceAccount | quote }}
resourceGroup: {{ .Requirements.velero.resourceGroup | quote }}
{{- else if eq .Requirements.cluster.provider "iks" }}
initContainers:
- name: velero-plugin-for-aws
Expand Down

0 comments on commit cff0d67

Please sign in to comment.