Skip to content

feat: default site backups - #672

Merged
AyshaHakeem merged 8 commits into
frappe:developfrom
AyshaHakeem:feat/default-site-backups
Oct 21, 2026
Merged

AyshaHakeem merged 8 commits into
frappe:developfrom
AyshaHakeem:feat/default-site-backups

Conversation

@AyshaHakeem

@AyshaHakeem AyshaHakeem commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor
Site state Result
No backup config (new site, restored site, or an existing site with no policy) Default daily GFS through the bench's hourly job
Custom GFS or FIFO saved in the dialog or the API Its own cron schedule and retention; the bench job skips it
User turned automated backups off Manual backups only

Patch: it enrolls only sites with no backup_retention record. Sites with a custom schedule keep it.

@greptile-apps

greptile-apps Bot commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

[High impact] The PR appears safe to merge; no actionable new issues remain.

Reviews (4) · Last reviewed commit: "test: Refactor tests" · Reviewed by Greptile

Comment thread pilot/core/site/backups.py Outdated
Add a calendar_rotation flag to backup retention. With it, every run is kept
for 7 days, Sunday runs for 28 days, runs on the 1st of a month for 366 days
and January 1 runs for 3653 days. The newest run is always kept.
New sites, and sites restored from a backup, now get daily backups with
calendar GFS retention unless the user sets a custom schedule. A custom
schedule replaces the default.

One hourly cron job per bench reads the audit log once and queues a backup
for each default site with no successful backup in 23 hours. The first backup
of a day includes files. A site is skipped for the day after six failures.
The sixth failed backup since the last successful one adds a "Backups of
<site> are failing" error to the notification feed.
A post-update patch enrolls each existing site that has no backup schedule
in the default backups. Sites with a custom schedule keep it.
The Backups page shows "Daily, with the default retention policy." for a site
on the default backups. The config dialog opens enabled with the default
retention, so saving it sets a custom schedule.
@AyshaHakeem
AyshaHakeem force-pushed the feat/default-site-backups branch from 11a1412 to c31bc79 Compare October 21, 2026 08:40
@AyshaHakeem

Copy link
Copy Markdown
Contributor Author

@greptileai retrigger confidence score check

@AyshaHakeem
AyshaHakeem merged commit 642a77d into frappe:develop Oct 21, 2026
6 checks passed
@AyshaHakeem

Copy link
Copy Markdown
Contributor Author

closes #625

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.

1 participant