-
Notifications
You must be signed in to change notification settings - Fork 16.4k
Open
Labels
area:corekind:taskA task that needs to be completed as part of a larger issueA task that needs to be completed as part of a larger issue
Description
Apache Airflow version
2.9.0
What happened?
There is a part of code that may have been dead since airflow 2.0.0 for task logs prefix. The code is extending a StreamHandler which is not being used anywhere. The current local or remote task logs are extending FileTaskHandler.
What you think should happen instead?
Either this should be completely removed from Airflow or fix the feature with FileTaskHandler
How to reproduce
Set AIRFLOW__LOGGING__TASK_LOG_PREFIX_TEMPLATE with an example provided in the docs.
Operating System
Any
Versions of Apache Airflow Providers
Not Applicable.
Deployment
Docker-Compose
Deployment details
No response
Anything else?
I have raised a PR #38709 in an attempt to make this feature working with FileTaskHandler.
Though I understand there might not be any point in bring a dead feature back.
So, I'm proposing to remove the config task_log_prefix_template all together residing in below code:
| class TaskHandlerWithCustomFormatter(logging.StreamHandler): |
Are you willing to submit PR?
- Yes I am willing to submit a PR!
Code of Conduct
- I agree to follow this project's Code of Conduct
Metadata
Metadata
Assignees
Labels
area:corekind:taskA task that needs to be completed as part of a larger issueA task that needs to be completed as part of a larger issue