Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

log retention env var name misleading #264

Open
jaketf opened this issue May 27, 2021 · 0 comments
Open

log retention env var name misleading #264

jaketf opened this issue May 27, 2021 · 0 comments
Assignees

Comments

@jaketf
Copy link

jaketf commented May 27, 2021

RETENTION=${ASTRONOMER__AIRFLOW__WORKER_LOG_RETENTION_DAYS:-15}

This environment variable implies that the retention is only relevant to the worker logs. however this script is also used to clean the scheduler logs in the helm chart. These should either be controlled by separate variables or this name should change to be ASTRONOMER__AIRFLOW__LOG_RETENTION_DAYS

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants