Add minute-level log retention for log groomer | issue #61814#61859
Add minute-level log retention for log groomer | issue #61814#61859MiteshSahu wants to merge 2 commits into
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 Contributors' Guide (https://github.com/apache/airflow/blob/main/contributing-docs/README.rst)
|
|
duplication with #61853 |
|
@MiteshSahu This PR has been converted to draft because it does not yet meet our Pull Request quality criteria. Issues found:
What to do next:
Converting a PR to draft is not a rejection — it is an invitation to bring the PR up to the project's standards so that maintainer review time is spent productively. If you have questions, feel free to ask on the Airflow Slack. Note: This comment was drafted by an AI-assisted triage tool and may contain mistakes. Once you have addressed the points above, an Apache Airflow maintainer — a real person — will take the next look at your PR. We use this two-stage triage process so that our maintainers' limited time is spent where it matters most: the conversation with you. |
|
This PR is obsolete now since PR #61855 got merged |
Description
Add minute-level log retention support for log groomer sidecar containers to enable sub-day log rotation in remote logging scenarios (e.g., OpenTelemetry FileLog Receiver).
Changes:
AIRFLOW__LOG_RETENTION_MINUTESenvironment variable (takes precedence overAIRFLOW__LOG_RETENTION_DAYS)logGroomerSidecar.retentionMinutesconfiguration option for all deployment types (workers, scheduler, triggerer, dag-processor)find -mminwhen minutes are configured, fallback to-mtime(days) otherwiseType of change
Testing
airflow-core/tests/unit/charts/log_groomer.pyhelm-tests/tests/chart_utils/log_groomer.pyRelated Issue
Closes #61814