Skip to content

Commit

Permalink
dont include git-sync containers in webserver for airflow 2.0
Browse files Browse the repository at this point in the history
Signed-off-by: Mathew Wicks <[email protected]>
  • Loading branch information
thesuperzapper committed Apr 20, 2021
1 parent 1b19f42 commit ca68e31
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions charts/airflow/templates/webserver/webserver-deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,8 @@ spec:
{{- end }}
{{- include "airflow.init_container.check_db" . | indent 8 }}
{{- include "airflow.init_container.wait_for_db_migrations" . | indent 8 }}
{{- if .Values.dags.gitSync.enabled }}
{{- if and (.Values.dags.gitSync.enabled) (.Values.airflow.legacyCommands) }}
{{- /* after 2.0, webserver no longer needs the DAG files, due to DAG Serialization */ -}}
{{- include "airflow.container.git_sync" (dict "Release" .Release "Values" .Values "sync_one_time" "true") | indent 8 }}
{{- end }}
containers:
Expand Down Expand Up @@ -138,7 +139,8 @@ spec:
mountPath: /opt/airflow/webserver_config.py
subPath: webserver_config.py
readOnly: true
{{- if .Values.dags.gitSync.enabled }}
{{- if and (.Values.dags.gitSync.enabled) (.Values.airflow.legacyCommands) }}
{{- /* after 2.0, webserver no longer needs the DAG files, due to DAG Serialization */ -}}
{{- include "airflow.container.git_sync" . | indent 8 }}
{{- end }}
{{- if .Values.airflow.extraContainers }}
Expand Down

0 comments on commit ca68e31

Please sign in to comment.