From 68d034979357aecfecbe2d5d01564eb000e1ee32 Mon Sep 17 00:00:00 2001 From: Rajesh Rajendran Date: Sun, 29 Dec 2019 13:53:52 +0530 Subject: [PATCH] Include annotations in templating. for example, If I have istio enabled in a namespace, and I want to deploy reloader in that, It doesn't make sense to include envoy there. So there should be an option to add `sidecar.istio.io/inject: "false"` annotation. --- .../kubernetes/chart/reloader/templates/deployment.yaml | 4 ++++ deployments/kubernetes/chart/reloader/values.yaml | 2 ++ deployments/kubernetes/templates/chart/values.yaml.tmpl | 2 ++ 3 files changed, 8 insertions(+) diff --git a/deployments/kubernetes/chart/reloader/templates/deployment.yaml b/deployments/kubernetes/chart/reloader/templates/deployment.yaml index e6a5bad8..c03a86f5 100644 --- a/deployments/kubernetes/chart/reloader/templates/deployment.yaml +++ b/deployments/kubernetes/chart/reloader/templates/deployment.yaml @@ -26,6 +26,10 @@ spec: {{- end }} template: metadata: +{{- if .Values.reloader.deployment.pod.annotations }} + annotations: +{{ toYaml .Values.reloader.deployment.pod.annotations | indent 8 }} +{{- end }} labels: {{ include "reloader-labels.chart" . | indent 8 }} {{- if .Values.reloader.deployment.labels }} diff --git a/deployments/kubernetes/chart/reloader/values.yaml b/deployments/kubernetes/chart/reloader/values.yaml index ff38a89b..e713e530 100644 --- a/deployments/kubernetes/chart/reloader/values.yaml +++ b/deployments/kubernetes/chart/reloader/values.yaml @@ -67,6 +67,8 @@ reloader: # cpu: "10m" # memory: "128Mi" resources: {} + pod: + annotations: {} rbac: enabled: true diff --git a/deployments/kubernetes/templates/chart/values.yaml.tmpl b/deployments/kubernetes/templates/chart/values.yaml.tmpl index 37c0cda7..d6d74a99 100644 --- a/deployments/kubernetes/templates/chart/values.yaml.tmpl +++ b/deployments/kubernetes/templates/chart/values.yaml.tmpl @@ -67,6 +67,8 @@ reloader: # cpu: "10m" # memory: "128Mi" resources: {} + pod: + annotations: {} rbac: enabled: true