Wastelander777 commented on issue #61814: URL: https://github.com/apache/airflow/issues/61814#issuecomment-3896751967
Hi! I have created a PR to address this issue. I think it's has better flexibility and safety than previous proposals: - By defaulting `AIRFLOW__LOG_RETENTION_MINUTES` to 0, the script automatically falls back to the original daily retention logic. Users can toggle between minutes and days without breaking the cleanup cycle. - It preserves the existing script structure, including the logs. I have verified that there is not documentation talking about AIRFLOW__LOG_RETENTION_DAYS, but I'm happy to add a section for this new variable to ensure it's discoverable for Docker users. Also, if the maintainers agree with this logic, I am more than happy to extend this to the Helm Chart templates and update the relevant tests to keep everything in sync. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
