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

feat: add log-cleanup sidecar to scheduler/worker #554

Merged

Conversation

thesuperzapper
Copy link
Member

What issues does your PR fix?

What does your PR do?

  • Adds a log-cleanup sidecar to the scheduler that automatically deletes old log files (by last-modified time)
  • Adds the following new values:
    • scheduler
      • scheduler.logCleanup.enabled (default: true)
      • scheduler.logCleanup.resources (default: {})
      • scheduler.logCleanup.retentionMinutes (default: 21600 = 15 days)
      • scheduler.logCleanup.intervalSeconds (default: 900 = 15 minutes)
    • workers
      • workers.logCleanup.enabled (default: true)
      • workers.logCleanup.resources (default: {})
      • workers.logCleanup.retentionMinutes (default: 21600 = 15 days)
      • workers.logCleanup.intervalSeconds (default: 900 = 15 minutes)

Checklist

For all Pull Requests

For releasing ONLY

Signed-off-by: Mathew Wicks <thesuperzapper@users.noreply.github.com>
@thesuperzapper thesuperzapper added the status/ready-to-merge status - this will be merged into next release label Mar 31, 2022
@thesuperzapper thesuperzapper added this to Unsorted in Issue Triage and PR Tracking via automation Mar 31, 2022
@thesuperzapper thesuperzapper added this to the airflow-8.6.0 milestone Mar 31, 2022
@thesuperzapper thesuperzapper moved this from Unsorted to PR | Ready to Merge in Issue Triage and PR Tracking Mar 31, 2022
@thesuperzapper thesuperzapper merged commit 4fbb856 into airflow-helm:main Mar 31, 2022
Issue Triage and PR Tracking automation moved this from PR | Ready to Merge to Done Mar 31, 2022
@thesuperzapper thesuperzapper deleted the log-cleanup-sidecar branch March 31, 2022 20:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status/ready-to-merge status - this will be merged into next release
Development

Successfully merging this pull request may close these issues.

add log-cleanup sidecar to scheduler/worker pods
1 participant