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/12/21 14:29:45 UTC

[GitHub] [airflow] KevYuen opened a new pull request #13221: Fix Default Config Jinja Template Error

KevYuen opened a new pull request #13221:
URL: https://github.com/apache/airflow/pull/13221


   The default param for these causes jinja to error due to having too many brackets.
   
   Users who copy the default configuration will get the following error on startup:
   ```
   jinja2.exceptions.TemplateSyntaxError: expected token ':', got '}'
   During handling of the above exception, another exception occurred:
   Traceback (most recent call last):
     File "/usr/local/bin/airflow", line 5, in <module>
       from airflow.__main__ import main
     File "/usr/local/lib/python3.6/site-packages/airflow/__init__.py", line 46, in <module>
       settings.initialize()
     File "/usr/local/lib/python3.6/site-packages/airflow/settings.py", line 432, in initialize
       LOGGING_CLASS_PATH = configure_logging()
     File "/usr/local/lib/python3.6/site-packages/airflow/logging_config.py", line 62, in configure_logging
       raise e
     File "/usr/local/lib/python3.6/site-packages/airflow/logging_config.py", line 57, in configure_logging
       dictConfig(logging_config)
     File "/usr/lib64/python3.6/logging/config.py", line 802, in dictConfig
       dictConfigClass(config).configure()
     File "/usr/lib64/python3.6/logging/config.py", line 573, in configure
       '%r: %s' % (name, e))
   ValueError: Unable to configure handler 'processor': expected token ':', got '}'
   ```


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

[GitHub] [airflow] mik-laj commented on pull request #13221: Fix Default Config Jinja Template Error

Posted by GitBox <gi...@apache.org>.
mik-laj commented on pull request #13221:
URL: https://github.com/apache/airflow/pull/13221#issuecomment-749072613


   This file is copied automatically on first startup and is used as a template.
   https://github.com/apache/airflow/blob/a3cc78dc16bf228901ba64e65d202d4685bf4fc7/airflow/configuration.py#L749-L757


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

[GitHub] [airflow] KevYuen commented on pull request #13221: Fix Default Config Jinja Template Error

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


   Looks like this is intended behaviour, closing!


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

[GitHub] [airflow] KevYuen edited a comment on pull request #13221: Fix Default Config Jinja Template Error

Posted by GitBox <gi...@apache.org>.
KevYuen edited a comment on pull request #13221:
URL: https://github.com/apache/airflow/pull/13221#issuecomment-749109623


   @mik-laj So if I understand corerctly, because of that the jinja template needs to be double escaped. 
   
   So users who manually copy over the configuration rather rely on the first start process to get the configuration file gets the double escaped values and thus encounter the jinja template parser issue. 


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

[GitHub] [airflow] KevYuen closed pull request #13221: Fix Default Config Jinja Template Error

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


   


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

[GitHub] [airflow] KevYuen commented on pull request #13221: Fix Default Config Jinja Template Error

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


   @mik-laj So if I understand corerctly, because of that the jinja template needs to be double escaped. 
   
   So users who manually copy over the files rather rely on the first start process to get the configuration file gets the double escaped values and thus encounter the jinja template parser issue. 


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