Change alembic logging level to WARN#30563
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/main/CONTRIBUTING.rst)
|
|
Do not use WARN. Use WARNING instead. If we agree to do this. |
hussein-awala
left a comment
There was a problem hiding this comment.
Currently the alembic logging configuration are loaded from this file without any possibility to update them:
Line 728 in 73e0313
I think it might be a good feature if we add a new Airflow configuration to provide a custom ini file for alembic, then you can update it as you like in your Airflow server. WDYT?
|
Yeah, I think that's a more complete solution, and would certainly resolve this for me. |
|
What about setting up |
Agree with @hussein-awala - being able to specify alembic.ini would be much better. |
|
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 5 days if no further activity occurs. Thank you for your contributions. |
|
the issue was closed by #31415 |
closes: #30562
^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an 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 a newsfragment file, named
{pr_number}.significant.rstor{issue_number}.significant.rst, in newsfragments.