Fix the broken template of default_airflow#13983
Fix the broken template of default_airflow#13983CastielWong wants to merge 1 commit intoapache:masterfrom CastielWong:master
Conversation
|
Congratulations on your first Pull Request and welcome to the Apache Airflow community! If you have any issues or are unsure about any anything please check our Contribution Guide (https://github.com/apache/airflow/blob/master/CONTRIBUTING.rst)
|
|
The Workflow run is cancelling this PR. Building images for the PR has failed. Follow the the workflow link to check the reason. |
|
The Workflow run is cancelling this PR. Building images for the PR has failed. Follow the the workflow link to check the reason. |
|
The Workflow run is cancelling this PR. Building images for the PR has failed. Follow the the workflow link to check the reason. |
|
The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Backport packages$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*. |
|
Looks like this issue is caused by "pylint", can't fix it without reconfig pre-commit thus close the PR. |
I tried to apply "dockerrize" to render the "default_airflow.cfg" template for my Airflow, yet Airflow failed to start with the new config file.
It spent me quite a while to figure out what had happened. When I double checked the "airflow.cfg" auto-generated by
airflow db init, I finally realized the reason why it failed was because of those extra brackets for delims in the template.So this PR is to fix the issue by removing those extra brackets in the template.
^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.