You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/08/19 09:13:50 UTC

[GitHub] [airflow] malthe opened a new issue, #25813: Helm chart should provide log rotation

malthe opened a new issue, #25813:
URL: https://github.com/apache/airflow/issues/25813

   ### Description
   
   The Helm chart should provide log rotation functionality for the various services in order to periodically clean up logs.
   
   ### Use case/motivation
   
   While log rotation for workers can be implemented using a DAG, it fits more naturally on the operating system level.
   
   ### Related issues
   
   _No response_
   
   ### Are you willing to submit a PR?
   
   - [X] Yes I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [X] I agree to follow this project's [Code of Conduct](https://github.com/apache/airflow/blob/main/CODE_OF_CONDUCT.md)
   


-- 
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-unsubscribe@airflow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [airflow] potiuk closed issue #25813: Helm chart should provide log rotation

Posted by GitBox <gi...@apache.org>.
potiuk closed issue #25813: Helm chart should provide log rotation
URL: https://github.com/apache/airflow/issues/25813


-- 
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-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [airflow] potiuk commented on issue #25813: Helm chart should provide log rotation

Posted by GitBox <gi...@apache.org>.
potiuk commented on issue #25813:
URL: https://github.com/apache/airflow/issues/25813#issuecomment-1220905764

   It already does. https://airflow.apache.org/docs/helm-chart/stable/parameters-ref.html (look for logGroomer). Maybe it needs to be surfaced better (feel free to make PR with docs update @malthe. but I believe it's all there. It could have more features for sure, but I think what is there is quite enough ? 
   
   Converting into discussion in case there is more to it that I see 


-- 
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-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org