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

Only keep 10 days of forgejo backups locally #228

Merged
merged 1 commit into from
Jan 7, 2025
Merged

Conversation

acrn
Copy link
Contributor

@acrn acrn commented Dec 18, 2024

The disk was full this morning resulting in internal server error from forgejo

@acrn acrn requested a review from theseal December 18, 2024 12:47
@acrn acrn force-pushed the acrn-mindre-backup branch from 9b4373c to 7aa3073 Compare December 18, 2024 13:02
@acrn
Copy link
Contributor Author

acrn commented Dec 18, 2024

I also wanted to change the cron trigger from */2 to 6-18/2 but an alert would fire if there is no backup newer than 3 hours. Only keeping 10 days of backups will solve the immediate problem anyway.

@acrn acrn changed the title Only keep 10 days of forgejo backups locally, only run backups during daytime Only keep 10 days of forgejo backups locally Jan 7, 2025
Copy link
Contributor

@mikaelfrykholm mikaelfrykholm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@acrn acrn merged commit 3ce36a1 into testing Jan 7, 2025
2 checks passed
@acrn acrn deleted the acrn-mindre-backup branch January 7, 2025 08:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants