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 intoapache:mainfrom
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. |
|
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