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

Pull backup:clean settings into config #16031

Merged
merged 1 commit into from
Jan 7, 2025
Merged

Conversation

snipe
Copy link
Owner

@snipe snipe commented Jan 7, 2025

This just pulls the hard-coded backup:clean settings into the env, using the same defaults that we've always used if you don't have these set.

Copy link

what-the-diff bot commented Jan 7, 2025

PR Summary

  • Introduction of New Environment Variables for Backup Retention
    These environment variables were added to .env.example. They allow for the customization of the retention period for different types of backups. The variable names and their default values are as follows:

    • ALL_BACKUP_KEEP_DAYS=7: Keeps all backups for 7 days.
    • DAILY_BACKUP_KEEP_DAYS=16: Keeps daily backups for 16 days.
    • WEEKLY_BACKUP_KEEP_WEEKS=8: Keeps weekly backups for 8 weeks.
    • MONTHLY_BACKUP_KEEP_MONTHS=4: Keeps monthly backups for 4 months.
    • YEARLY_BACKUP_KEEP_YEARS=2: Keeps yearly backups for 2 years.
    • BACKUP_PURGE_OLDEST_AT_MEGS=5000: Initiates a purge of the oldest backups when the total backup size exceeds 5000 megabytes.
  • Configuration File Updated for Backup Retention Period
    The configuration file config/backup.php was updated to use the new environment variables rather than the previous hardcoded values. This allows for more flexibility, as the retention periods can be adjusted more easily based on individual needs or requirements.

@snipe snipe merged commit 407d69b into develop Jan 7, 2025
9 checks passed
@snipe snipe deleted the make_backups_more_configurable branch January 7, 2025 11:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant