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/12/27 22:43:50 UTC

[GitHub] [airflow] dstandish opened a new pull request #20529: Config patches (for tests) should patch before resolving

dstandish opened a new pull request #20529:
URL: https://github.com/apache/airflow/pull/20529


   These utilities exist to patch airflow config. In some cases, such as the default timezone, importing a module such a airflow.settings resolves the airflow config before it can be patched by the context manager.  This change imports locally, as late as possible, to reduce the likelihood of problems like this.  In the case of the `env_vars` patcher, we no longer import airflow at all, so using this patcher will always work.
   


-- 
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-unsubscribe@airflow.apache.org

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



[GitHub] [airflow] dstandish closed pull request #20529: Config patches (for tests) should patch before resolving

Posted by GitBox <gi...@apache.org>.
dstandish closed pull request #20529:
URL: https://github.com/apache/airflow/pull/20529


   


-- 
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-unsubscribe@airflow.apache.org

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



[GitHub] [airflow] dstandish commented on pull request #20529: Config patches (for tests) should patch before resolving

Posted by GitBox <gi...@apache.org>.
dstandish commented on pull request #20529:
URL: https://github.com/apache/airflow/pull/20529#issuecomment-1001799700


   hmmm.....  this still doesn't seem to let us patch TIMEZONE


-- 
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-unsubscribe@airflow.apache.org

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