[PR] Avoid log-groomer container failures when log persistence is enabled [airflow]

2024-02-07 Thread via GitHub
arovira opened a new pull request, #37222: URL: https://github.com/apache/airflow/pull/37222 --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#pu

Re: [PR] Avoid log-groomer container failures when log persistence is enabled [airflow]

2024-02-07 Thread via GitHub
boring-cyborg[bot] commented on PR #37222: URL: https://github.com/apache/airflow/pull/37222#issuecomment-1931977891 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'

Re: [PR] Avoid log-groomer container failures when log persistence is enabled [airflow]

2024-02-07 Thread via GitHub
potiuk commented on code in PR #37222: URL: https://github.com/apache/airflow/pull/37222#discussion_r1481603016 ## Dockerfile: ## @@ -1181,10 +1181,14 @@ echo "Cleaning logs every $EVERY seconds" while true; do echo "Trimming airflow logs to ${RETENTION} days." - find "${

Re: [PR] Avoid log-groomer container failures when log persistence is enabled [airflow]

2024-02-07 Thread via GitHub
arovira commented on code in PR #37222: URL: https://github.com/apache/airflow/pull/37222#discussion_r1481610381 ## Dockerfile: ## @@ -1181,10 +1181,14 @@ echo "Cleaning logs every $EVERY seconds" while true; do echo "Trimming airflow logs to ${RETENTION} days." - find "$

Re: [PR] Avoid log-groomer container failures when log persistence is enabled [airflow]

2024-02-07 Thread via GitHub
potiuk merged PR #37222: URL: https://github.com/apache/airflow/pull/37222 -- 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: commits-unsubscr...@airflow.a

Re: [PR] Avoid log-groomer container failures when log persistence is enabled [airflow]

2024-02-07 Thread via GitHub
boring-cyborg[bot] commented on PR #37222: URL: https://github.com/apache/airflow/pull/37222#issuecomment-1932573485 Awesome work, congrats on your first merged pull request! You are invited to check our [Issue Tracker](https://github.com/apache/airflow/issues) for additional contributions.