jedcunningham commented on code in PR #29941:
URL: https://github.com/apache/airflow/pull/29941#discussion_r1129838941
##########
chart/templates/jobs/migrate-database-job.yaml:
##########
@@ -96,37 +91,34 @@ spec:
args: {{ tpl (toYaml .Values.migrateDatabaseJob.args) . | nindent 12
}}
{{- end }}
{{- if .Values.migrateDatabaseJob.applyCustomEnv }}
- envFrom:
- {{- include "custom_airflow_environment_from" . | default "\n []" |
indent 10 }}
- env:
- {{- include "custom_airflow_environment" . | indent 10 }}
+ envFrom: {{- include "custom_airflow_environment_from" . | default
"\n []" | indent 10 }}
+ env: {{- include "custom_airflow_environment" . | indent 10 }}
Review Comment:
And these?
##########
chart/templates/jobs/create-user-job.yaml:
##########
@@ -96,36 +91,33 @@ spec:
args: {{ tpl (toYaml .Values.createUserJob.args) . | nindent 12 }}
{{- end }}
{{- if .Values.createUserJob.applyCustomEnv }}
- envFrom:
- {{- include "custom_airflow_environment_from" . | default "\n []" |
indent 10 }}
- env:
- {{- include "custom_airflow_environment" . | indent 10 }}
+ envFrom: {{- include "custom_airflow_environment_from" . | default
"\n []" | indent 10 }}
+ env: {{- include "custom_airflow_environment" . | indent 10 }}
Review Comment:
Do these need to be nindent instead?
##########
chart/templates/webserver/webserver-deployment.yaml:
##########
@@ -127,37 +123,34 @@ spec:
initContainers:
{{- if .Values.webserver.waitForMigrations.enabled }}
- name: wait-for-airflow-migrations
- resources:
-{{ toYaml .Values.webserver.resources | indent 12 }}
+ resources: {{- toYaml .Values.webserver.resources | nindent 12 }}
image: {{ template "airflow_image_for_migrations" . }}
imagePullPolicy: {{ .Values.images.airflow.pullPolicy }}
volumeMounts:
-{{- include "airflow_config_mount" . | nindent 12 }}
-{{- if .Values.volumeMounts }}
-{{- toYaml .Values.volumeMounts | nindent 12 }}
-{{- end }}
-{{- if .Values.webserver.extraVolumeMounts }}
-{{ toYaml .Values.webserver.extraVolumeMounts | indent 12 }}
-{{- end }}
-{{- if or .Values.webserver.webserverConfig
.Values.webserver.webserverConfigConfigMapName }}
-{{ include "airflow_webserver_config_mount" . | indent 12 }}
-{{- end }}
- args:
- {{- include "wait-for-migrations-command" . | indent 10 }}
- envFrom:
- {{- include "custom_airflow_environment_from" . | default "\n []" |
indent 10 }}
+ {{- include "airflow_config_mount" . | nindent 12 }}
+ {{- if .Values.volumeMounts }}
+ {{- toYaml .Values.volumeMounts | nindent 12 }}
+ {{- end }}
+ {{- if .Values.webserver.extraVolumeMounts }}
+ {{- toYaml .Values.webserver.extraVolumeMounts | nindent 12 }}
+ {{- end }}
+ {{- if or .Values.webserver.webserverConfig
.Values.webserver.webserverConfigConfigMapName }}
+ {{- include "airflow_webserver_config_mount" . | nindent 12 }}
+ {{- end }}
+ args: {{- include "wait-for-migrations-command" . | indent 10 }}
+ envFrom: {{- include "custom_airflow_environment_from" . | default
"\n []" | indent 10 }}
Review Comment:
These?
##########
chart/templates/scheduler/scheduler-deployment.yaml:
##########
@@ -178,12 +168,11 @@ spec:
{{- if .Values.scheduler.args }}
args: {{ tpl (toYaml .Values.scheduler.args) . | nindent 12 }}
{{- end }}
- envFrom:
- {{- include "custom_airflow_environment_from" . | default "\n []" |
indent 10 }}
+ envFrom: {{- include "custom_airflow_environment_from" . | default
"\n []" | indent 10 }}
Review Comment:
This one?
##########
chart/templates/webserver/webserver-deployment.yaml:
##########
@@ -167,77 +160,75 @@ spec:
command: {{ tpl (toYaml .Values.webserver.command) . | nindent 12 }}
{{- end }}
{{- if .Values.webserver.args }}
- args: {{ tpl (toYaml .Values.webserver.args) . | nindent 12 }}
+ args: {{- tpl (toYaml .Values.webserver.args) . | nindent 12 }}
{{- end }}
- resources:
-{{ toYaml .Values.webserver.resources | indent 12 }}
+ resources: {{- toYaml .Values.webserver.resources | nindent 12 }}
volumeMounts:
-{{- if semverCompare ">=1.10.12" .Values.airflowVersion }}
+ {{- if semverCompare ">=1.10.12" .Values.airflowVersion }}
- name: config
mountPath: {{ include "airflow_pod_template_file" .
}}/pod_template_file.yaml
subPath: pod_template_file.yaml
readOnly: true
-{{- end }}
-{{- include "airflow_config_mount" . | nindent 12 }}
-{{- if or .Values.webserver.webserverConfig
.Values.webserver.webserverConfigConfigMapName }}
-{{ include "airflow_webserver_config_mount" . | indent 12 }}
-{{- end }}
-{{- if and (semverCompare "<2.0.0" .Values.airflowVersion) (or
.Values.dags.gitSync.enabled .Values.dags.persistence.enabled) }}
- {{- include "airflow_dags_mount" . | nindent 12 }}
-{{- end }}
-{{- if .Values.logs.persistence.enabled }}
+ {{- end }}
+ {{- include "airflow_config_mount" . | nindent 12 }}
+ {{- if or .Values.webserver.webserverConfig
.Values.webserver.webserverConfigConfigMapName }}
+ {{- include "airflow_webserver_config_mount" . | nindent 12 }}
+ {{- end }}
+ {{- if and (semverCompare "<2.0.0" .Values.airflowVersion) (or
.Values.dags.gitSync.enabled .Values.dags.persistence.enabled) }}
+ {{- include "airflow_dags_mount" . | nindent 12 }}
+ {{- end }}
+ {{- if .Values.logs.persistence.enabled }}
- name: logs
mountPath: {{ template "airflow_logs" . }}
-{{- end }}
-{{- if .Values.volumeMounts }}
-{{- toYaml .Values.volumeMounts | nindent 12 }}
-{{- end }}
-{{- if .Values.webserver.extraVolumeMounts }}
-{{ toYaml .Values.webserver.extraVolumeMounts | indent 12 }}
-{{- end }}
+ {{- end }}
+ {{- if .Values.volumeMounts }}
+ {{- toYaml .Values.volumeMounts | nindent 12 }}
+ {{- end }}
+ {{- if .Values.webserver.extraVolumeMounts }}
+ {{- toYaml .Values.webserver.extraVolumeMounts | nindent 12 }}
+ {{- end }}
ports:
- name: airflow-ui
containerPort: {{ .Values.ports.airflowUI }}
livenessProbe:
httpGet:
path: {{if .Values.config.webserver.base_url }}{{- with urlParse
(tpl .Values.config.webserver.base_url .) }}{{ .path }}{{end}}{{end}}/health
port: {{ .Values.ports.airflowUI }}
-{{- if .Values.config.webserver.base_url}}
+ {{- if .Values.config.webserver.base_url}}
httpHeaders:
- name: Host
value: {{ regexReplaceAll ":\\d+$" (urlParse (tpl
.Values.config.webserver.base_url .)).host "" }}
-{{- end }}
+ {{- end }}
scheme: {{ .Values.webserver.livenessProbe.scheme | default
"http" }}
initialDelaySeconds: {{
.Values.webserver.livenessProbe.initialDelaySeconds | default 15 }}
timeoutSeconds: {{ .Values.webserver.livenessProbe.timeoutSeconds
| default 30 }}
failureThreshold: {{
.Values.webserver.livenessProbe.failureThreshold | default 20 }}
periodSeconds: {{ .Values.webserver.livenessProbe.periodSeconds |
default 5 }}
readinessProbe:
httpGet:
- path: {{if .Values.config.webserver.base_url }}{{- with urlParse
(tpl .Values.config.webserver.base_url .) }}{{ .path }}{{end}}{{end}}/health
+ path: {{ if .Values.config.webserver.base_url }}{{- with
urlParse (tpl .Values.config.webserver.base_url .) }}{{ .path }}{{ end }}{{ end
}}/health
port: {{ .Values.ports.airflowUI }}
-{{- if .Values.config.webserver.base_url}}
+ {{- if .Values.config.webserver.base_url }}
httpHeaders:
- name: Host
value: {{ regexReplaceAll ":\\d+$" (urlParse (tpl
.Values.config.webserver.base_url .)).host "" }}
-{{- end }}
+ {{- end }}
scheme: {{ .Values.webserver.readinessProbe.scheme | default
"http" }}
initialDelaySeconds: {{
.Values.webserver.readinessProbe.initialDelaySeconds | default 15 }}
timeoutSeconds: {{ .Values.webserver.readinessProbe.timeoutSeconds
| default 30 }}
failureThreshold: {{
.Values.webserver.readinessProbe.failureThreshold | default 20 }}
periodSeconds: {{ .Values.webserver.readinessProbe.periodSeconds |
default 5 }}
- envFrom:
- {{- include "custom_airflow_environment_from" . | default "\n []" |
indent 10 }}
+ envFrom: {{- include "custom_airflow_environment_from" . | default
"\n []" | indent 10 }}
Review Comment:
This?
##########
chart/templates/workers/worker-deployment.yaml:
##########
@@ -143,38 +135,35 @@ spec:
volumeMounts:
- name: logs
mountPath: {{ template "airflow_logs" . }}
- {{- end }}
+ {{- end }}
- name: wait-for-airflow-migrations
- resources:
-{{ toYaml .Values.workers.resources | indent 12 }}
+ resources: {{- toYaml .Values.workers.resources | nindent 12 }}
image: {{ template "airflow_image_for_migrations" . }}
imagePullPolicy: {{ .Values.images.airflow.pullPolicy }}
volumeMounts:
-{{- include "airflow_config_mount" . | nindent 12 }}
-{{- if .Values.volumeMounts }}
-{{- toYaml .Values.volumeMounts | nindent 12 }}
-{{- end }}
-{{- if .Values.workers.extraVolumeMounts }}
-{{ toYaml .Values.workers.extraVolumeMounts | indent 12 }}
-{{- end }}
-{{- if or .Values.webserver.webserverConfig
.Values.webserver.webserverConfigConfigMapName }}
-{{ include "airflow_webserver_config_mount" . | indent 12 }}
-{{- end }}
- args:
- {{- include "wait-for-migrations-command" . | indent 10 }}
- envFrom:
- {{- include "custom_airflow_environment_from" . | default "\n []" |
indent 10 }}
+ {{- include "airflow_config_mount" . | nindent 12 }}
+ {{- if .Values.volumeMounts }}
+ {{- toYaml .Values.volumeMounts | nindent 12 }}
+ {{- end }}
+ {{- if .Values.workers.extraVolumeMounts }}
+ {{- toYaml .Values.workers.extraVolumeMounts | nindent 12 }}
+ {{- end }}
+ {{- if or .Values.webserver.webserverConfig
.Values.webserver.webserverConfigConfigMapName }}
+ {{- include "airflow_webserver_config_mount" . | nindent 12 }}
+ {{- end }}
+ args: {{- include "wait-for-migrations-command" . | indent 10 }}
+ envFrom: {{- include "custom_airflow_environment_from" . | default
"\n []" | indent 10 }}
Review Comment:
These?
##########
chart/templates/scheduler/scheduler-deployment.yaml:
##########
@@ -135,37 +128,34 @@ spec:
initContainers:
{{- if .Values.scheduler.waitForMigrations.enabled }}
- name: wait-for-airflow-migrations
- resources:
-{{ toYaml .Values.scheduler.resources | indent 12 }}
+ resources: {{- toYaml .Values.scheduler.resources | nindent 12 }}
image: {{ template "airflow_image_for_migrations" . }}
imagePullPolicy: {{ .Values.images.airflow.pullPolicy }}
volumeMounts:
-{{- include "airflow_config_mount" . | nindent 12 }}
-{{- if .Values.volumeMounts }}
-{{- toYaml .Values.volumeMounts | nindent 12 }}
-{{- end }}
-{{- if .Values.scheduler.extraVolumeMounts }}
-{{ toYaml .Values.scheduler.extraVolumeMounts | indent 12 }}
-{{- end }}
-{{- if or .Values.webserver.webserverConfig
.Values.webserver.webserverConfigConfigMapName }}
-{{ include "airflow_webserver_config_mount" . | indent 12 }}
-{{- end }}
- args:
- {{- include "wait-for-migrations-command" . | indent 10 }}
- envFrom:
- {{- include "custom_airflow_environment_from" . | default "\n []" |
indent 10 }}
+ {{- include "airflow_config_mount" . | nindent 12 }}
+ {{- if .Values.volumeMounts }}
+ {{- toYaml .Values.volumeMounts | nindent 12 }}
+ {{- end }}
+ {{- if .Values.scheduler.extraVolumeMounts }}
+ {{- toYaml .Values.scheduler.extraVolumeMounts | nindent 12 }}
+ {{- end }}
+ {{- if or .Values.webserver.webserverConfig
.Values.webserver.webserverConfigConfigMapName }}
+ {{- include "airflow_webserver_config_mount" . | nindent 12 }}
+ {{- end }}
+ args: {{- include "wait-for-migrations-command" . | indent 10 }}
+ envFrom: {{- include "custom_airflow_environment_from" . | default
"\n []" | indent 10 }}
Review Comment:
And these?
##########
chart/templates/triggerer/triggerer-deployment.yaml:
##########
@@ -129,32 +123,30 @@ spec:
image: {{ template "airflow_image_for_migrations" . }}
imagePullPolicy: {{ .Values.images.airflow.pullPolicy }}
volumeMounts:
-{{- include "airflow_config_mount" . | nindent 12 }}
+ {{- include "airflow_config_mount" . | nindent 12 }}
{{- if .Values.volumeMounts }}
- {{- toYaml .Values.volumeMounts | nindent 12 }}
+ {{- toYaml .Values.volumeMounts | nindent 12 }}
{{- end }}
{{- if .Values.triggerer.extraVolumeMounts }}
- {{ toYaml .Values.triggerer.extraVolumeMounts | nindent 12 }}
+ {{- toYaml .Values.triggerer.extraVolumeMounts | nindent 12 }}
{{- end }}
-{{- if or .Values.webserver.webserverConfig
.Values.webserver.webserverConfigConfigMapName }}
-{{ include "airflow_webserver_config_mount" . | indent 12 }}
-{{- end }}
- args:
- {{- include "wait-for-migrations-command" . | nindent 10 }}
- envFrom:
- {{- include "custom_airflow_environment_from" . | default "\n []" |
nindent 10 }}
+ {{- if or .Values.webserver.webserverConfig
.Values.webserver.webserverConfigConfigMapName }}
+ {{- include "airflow_webserver_config_mount" . | nindent 12 }}
+ {{- end }}
+ args: {{- include "wait-for-migrations-command" . | indent 10 }}
+ envFrom: {{- include "custom_airflow_environment_from" . | default
"\n []" | indent 10 }}
Review Comment:
These?
##########
chart/templates/workers/worker-deployment.yaml:
##########
@@ -231,30 +219,29 @@ spec:
readOnly: true
{{- end }}
{{- if or .Values.dags.persistence.enabled
.Values.dags.gitSync.enabled }}
- {{- include "airflow_dags_mount" . | nindent 12 }}
+ {{- include "airflow_dags_mount" . | nindent 12 }}
{{- end }}
-{{- if or .Values.webserver.webserverConfig
.Values.webserver.webserverConfigConfigMapName }}
-{{ include "airflow_webserver_config_mount" . | indent 12 }}
-{{- end }}
- envFrom:
- {{- include "custom_airflow_environment_from" . | default "\n []" |
indent 10 }}
+ {{- if or .Values.webserver.webserverConfig
.Values.webserver.webserverConfigConfigMapName }}
+ {{- include "airflow_webserver_config_mount" . | nindent 12 }}
+ {{- end }}
+ envFrom: {{- include "custom_airflow_environment_from" . | default
"\n []" | indent 10 }}
Review Comment:
This?
##########
chart/templates/workers/worker-deployment.yaml:
##########
@@ -306,35 +291,34 @@ spec:
- name: kerberos-ccache
mountPath: {{ .Values.kerberos.ccacheMountPath | quote }}
readOnly: false
-{{- if .Values.volumeMounts }}
-{{- toYaml .Values.volumeMounts | nindent 12 }}
-{{- end }}
-{{- if .Values.workers.extraVolumeMounts }}
-{{ toYaml .Values.workers.extraVolumeMounts | indent 12 }}
-{{- end }}
-{{- if or .Values.webserver.webserverConfig
.Values.webserver.webserverConfigConfigMapName }}
-{{ include "airflow_webserver_config_mount" . | indent 12 }}
-{{- end }}
- envFrom:
- {{- include "custom_airflow_environment_from" . | default "\n []" |
indent 10 }}
+ {{- if .Values.volumeMounts }}
+ {{- toYaml .Values.volumeMounts | nindent 12 }}
+ {{- end }}
+ {{- if .Values.workers.extraVolumeMounts }}
+ {{- toYaml .Values.workers.extraVolumeMounts | nindent 12 }}
+ {{- end }}
+ {{- if or .Values.webserver.webserverConfig
.Values.webserver.webserverConfigConfigMapName }}
+ {{- include "airflow_webserver_config_mount" . | nindent 12 }}
+ {{- end }}
+ envFrom: {{- include "custom_airflow_environment_from" . | default
"\n []" | nindent 10 }}
Review Comment:
This?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
