jscheffl commented on code in PR #59875:
URL: https://github.com/apache/airflow/pull/59875#discussion_r2649916493
##########
airflow-core/src/airflow/cli/commands/backfill_command.py:
##########
@@ -38,7 +38,7 @@
@providers_configuration_loaded
def create_backfill(args) -> None:
"""Create backfill job or dry run for a DAG or list of DAGs using regex."""
- logging.basicConfig(level=settings.LOGGING_LEVEL,
format=settings.SIMPLE_LOG_FORMAT)
+ logging.basicConfig(level=logging.INFO, format=settings.SIMPLE_LOG_FORMAT)
Review Comment:
I also thought like this and forgot to add a not in advance. Was also
surprised that:
- In settings.py:110 (see
https://github.com/apache/airflow/pull/59875/changes#diff-3158396aa180e2e63b8b5662f49d2bf3625a0cb51c8c4c34d44692840de77784L110)
it is also statically set to `INFO`
- Nowhere in settings.py this is being adjusted during bootstrap - so it is
basically only a constant value
- Also via local_settings.py or any other means this constant is adjusted
Let me know if I over-look any part of code where this (constant) is defined
differently... then I am fine to adjust. With this change at least it is clear
that that logging config in here is just static `INFO`. Not good but the same
as before.
Might be a follow-up PR to make it better. This should be a 1:1 bad->similar
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
