Skip to content

Conversation

@justinpakzad
Copy link
Contributor


In the configure_logging function, the root logger level is hard coded to INFO. Since the user defined loggers inherit from the root, it wouldn't pick up the DEBUG level even if AIRFLOW__LOGGING__LOGGING_LEVEL=DEBUG is set.
Based on the conversation in #59634, it seems like it shouldn't be hard coded but I wasn't entirely sure if that was the conclusion.

related: #58965

Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

Copy link
Member

@potiuk potiuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am all for it - and it looks like it might some people problem - but maybe there was a reason for it -> @ashb @amoghrajesh ?

@potiuk potiuk added the backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch label Jan 20, 2026
@potiuk potiuk added this to the Airflow 3.1.7 milestone Jan 20, 2026
@justinpakzad justinpakzad force-pushed the fix-hardcoded-log-level-in-root branch from f9f8823 to 5aba24c Compare January 22, 2026 22:51
@potiuk potiuk merged commit f08d414 into apache:main Jan 23, 2026
61 checks passed
github-actions bot pushed a commit that referenced this pull request Jan 23, 2026
…evel (#60784)

* Use log_level for root logger instead of hardcoded INFO level

* Added test for debug level
(cherry picked from commit f08d414)

Co-authored-by: Justin Pakzad <114518232+justinpakzad@users.noreply.github.com>
@github-actions
Copy link

Backport successfully created: v3-1-test

Status Branch Result
v3-1-test PR Link

potiuk pushed a commit that referenced this pull request Jan 23, 2026
…evel (#60784) (#60970)

* Use log_level for root logger instead of hardcoded INFO level

* Added test for debug level
(cherry picked from commit f08d414)

Co-authored-by: Justin Pakzad <114518232+justinpakzad@users.noreply.github.com>
suii2210 pushed a commit to suii2210/airflow that referenced this pull request Jan 26, 2026
…#60784)

* Use log_level for root logger instead of hardcoded INFO level

* Added test for debug level
shreyas-dev pushed a commit to shreyas-dev/airflow that referenced this pull request Jan 29, 2026
…#60784)

* Use log_level for root logger instead of hardcoded INFO level

* Added test for debug level
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:logging backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants