You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2020/08/13 23:03:24 UTC

[GitHub] [airflow] alikhtag commented on pull request #10302: [BUGFIX] podMutations are now deployed correctly to airflow_local_settings.py

alikhtag commented on pull request #10302:
URL: https://github.com/apache/airflow/pull/10302#issuecomment-673751455


   > Why are we not just having podMutation as a string and allowing people to write whatever function they want? I don't want to start prescribing values or we'll maintain it forever.
   
   Hmm this yaml configuration of `podMutation` is very useful, since it will transform to json (or python dict), which is harder to write as string in this yaml, in my opinion. But `podMutation` can have a templating string ( using `tpl .values.podMutiation.pythonFunctionStr` ) as value as well. So if one decides to write function themselves they can overwrite it or use the yaml for templating. I will add this idea soon. 


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org