Chart: fix extra containers docs and remove krb excess if (#20787)

apache · Jan 14, 2022 · 5ca5693 · 5ca5693
1 parent bbe1eff
commit 5ca5693
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 7 deletions.
diff --git a/chart/templates/workers/worker-deployment.yaml b/chart/templates/workers/worker-deployment.yaml
@@ -277,15 +277,13 @@ spec:
 {{- if .Values.workers.extraVolumes }}
 {{ toYaml .Values.workers.extraVolumes | indent 8 }}
 {{- end }}
-        {{- if .Values.kerberos.enabled }}
-        - name: kerberos-keytab
-          secret:
-            secretName: {{ include "kerberos_keytab_secret" . | quote }}
-        {{- end }}
         - name: config
           configMap:
             name: {{ template "airflow_config" . }}
         {{- if .Values.kerberos.enabled }}
+        - name: kerberos-keytab
+          secret:
+            secretName: {{ include "kerberos_keytab_secret" . | quote }}
         - name: kerberos-ccache
           emptyDir: {}
         {{- end }}

diff --git a/docs/helm-chart/using-additional-containers.rst b/docs/helm-chart/using-additional-containers.rst
@@ -22,7 +22,7 @@ Sidecar Containers
 ------------------
 
 If you want to deploy your own sidecar container, you can add it through the ``extraContainers`` parameter.
-You can define different containers for the scheduler, webserver and worker pods.
+You can define different containers for the scheduler, webserver, worker, triggerer, flower, create user Job and migrate database Job Pods.
 
 For example, sidecars that sync DAGs from object storage.
 
@@ -49,7 +49,7 @@ Init Containers
 ---------------
 
 You can also deploy extra init containers through the ``extraInitContainers`` parameter.
-You can define different containers for the scheduler, webserver and worker pods.
+You can define different containers for the scheduler, webserver, worker and triggerer pods.
 
 For example, an init container that just says hello: