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 2021/06/10 18:29:09 UTC

[GitHub] [airflow] ashb commented on a change in pull request #16110: Added windows extensions

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




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