New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add podLabels to velero deployment template #66
Conversation
Signed-off-by: Joffrey Janiec <jjaniec@student.42.fr>
{{- if .Values.podLabels }} | ||
{{- toYaml .Values.podLabels | nindent 8 }} | ||
{{- end }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Recommends adding pod labels to restic daemonset also.
Signed-off-by: Joffrey Janiec <jjaniec@student.42.fr>
# Annotations to add to the Restic daemonset's pod template. Optional. | ||
# | ||
# If using kube2iam or kiam, use the following annotation with your AWS_ACCOUNT_ID | ||
# and RESTIC_ROLE_NAME filled in: | ||
# iam.amazonaws.com/role: arn:aws:iam::<AWS_ACCOUNT_ID>:role/<RESTIC_ROLE_NAME> | ||
podAnnotations: {} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's the difference to this https://github.com/vmware-tanzu/helm-charts/blob/1e74a1b/charts/velero/values.yaml#L12-L17?
The purpose is to separate the role of restic and velero?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @jjaniec this is looking good. Please address the question below and do a version bump so we can move forward with this!
The #120 addressed add podLabels to Velero deployment and restic DaemonSet, so close this one. |
…re-tanzu#66) * set rbac api version with template rbac.apiVersion Signed-off-by: Jorrit Salverda <jsalverda@travix.com> * bump chart version to 11.14.0 Signed-off-by: Jorrit Salverda <jsalverda@travix.com>
Added the ability to set own pod labels to velero deployment template
Signed-off-by: Joffrey Janiec jjaniec@student.42.fr