JKrehling commented on code in PR #41341:
URL: https://github.com/apache/airflow/pull/41341#discussion_r1821362746


##########
chart/templates/dag-processor/dag-processor-deployment.yaml:
##########
@@ -229,6 +231,10 @@ spec:
         {{- if .Values.dagProcessor.extraContainers }}
           {{- tpl (toYaml .Values.dagProcessor.extraContainers) . | nindent 8 
}}
         {{- end }}
+            - name: tmp
+              mountPath: /tmp 
+            - name: opt
+              mountPath: /opt/airflow                      

Review Comment:
   So this would be changed in the dag-processor, flower, scheduler, webserver, 
worker deployments. 
   
   There are some yaml tamplates though where /opt/airflow is hardcoded already 
so I believe I was following that.  For example if you look in 
basic_template.yaml 
   
   Im not certain if all files in the "dag-processor, flower, scheduler, 
webserver, worker deployments" pods are only created in the airflow_home 
directory, or if both should be mounted, /opt/airflow and whatever airflowhome 
is set to.  
   
   Let me know if that makes sense. 



-- 
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]

Reply via email to