ashb commented on a change in pull request #16110:
URL: https://github.com/apache/airflow/pull/16110#discussion_r649426106
##########
File path: airflow/__main__.py
##########
@@ -34,6 +35,15 @@ def main():
os.environ['KRB5CCNAME'] = conf.get('kerberos', 'ccache')
os.environ['KRB5_KTNAME'] = conf.get('kerberos', 'keytab')
+ # if dags folder has to be set to configured value, make sure it is set
properly (needed on Dask-Workers)
Review comment:
This is not the right fix for this.
There's a bug in a previous feature where the "dag folder" should be
replaced with `DAG_FOLDER` which is then automaticall replaced, but this isn't
working.
We should fix that rather than adding a new config and new way of making
this work.
See
https://github.com/apache/airflow/blob/9ba796ef40fe833aba58f5aa13a63587106d8ffd/airflow/utils/cli.py#L160-L167
for where the code is (the problem is on the command we send to the executor.)
https://github.com/apache/airflow/issues/8061
--
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:
[email protected]
