Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix stacklevel for TaskContextLogger #39142

Merged

Conversation

dstandish
Copy link
Contributor

Previously it would show the message as coming from context logger module and not the actual call site.
]

Previously it would show the message as coming from context logger module and not the actual call site.
@dstandish dstandish merged commit acc2338 into apache:main Apr 23, 2024
42 checks passed
@dstandish dstandish deleted the fix-stacklevel-for-taskcontextlogger branch April 23, 2024 03:02
RodrigoGanancia pushed a commit to RodrigoGanancia/airflow that referenced this pull request May 10, 2024
Previously it would show the message as coming from context logger module and not the actual call site.
@utkarsharma2 utkarsharma2 added the type:misc/internal Changelog: Misc changes that should appear in change log label Jun 3, 2024
@utkarsharma2 utkarsharma2 added this to the Airflow 2.9.2 milestone Jun 4, 2024
ephraimbuddy pushed a commit that referenced this pull request Jun 4, 2024
Previously it would show the message as coming from context logger module and not the actual call site.

(cherry picked from commit acc2338)
@ephraimbuddy ephraimbuddy added changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) and removed type:misc/internal Changelog: Misc changes that should appear in change log labels Aug 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:logging changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants