potiuk commented on PR #29624: URL: https://github.com/apache/airflow/pull/29624#issuecomment-1437263567
I think the idea in the #28800 was to be able to specify some parameters in the Helm Chart Values in a similar way for K8S as it si for celery. What it would likely mean - is merging the parameters specified in the values.yaml into the pod_template_file (providing defaults) and making it possible to run K8S executor even without a pod template file. The pod-template file is nice and super flexible, but as I imagine this one - it would be a nice way to configure most of the POD parameters in a very similar way for Celery and K8S pods, which seems to me like a good idea. It is kind of strange (byt justified in most complex cases) to have the parameters specified in pod_template_file. but having a static values (defaults?) specified in Helm chart would make it much easier to configure the installation for simple cases where pod template file flexibility is not really needed. @jedcunningham @dimberman @dstandish - WDYT about such a feature? This was - I think the original idea behind this feature (at least this is how I understood it). A -- 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: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
