Skip to content

[DEV-15000] Update Azure postgres clusters to use differential backups instead of incremental.#548

Merged
NathanOkolita merged 1 commit intomainfrom
dev-15000
Mar 13, 2026
Merged

[DEV-15000] Update Azure postgres clusters to use differential backups instead of incremental.#548
NathanOkolita merged 1 commit intomainfrom
dev-15000

Conversation

@NathanOkolita
Copy link
Contributor

@NathanOkolita NathanOkolita commented Mar 13, 2026

Note

Medium Risk
Changes the backup strategy and schedule for Azure Postgres clusters, which can affect restore behavior, storage usage, and backup/restore timelines if misconfigured.

Overview
Updates azure/application.tf to stop scheduling pgBackRest incremental backups and instead run daily differential backups at midnight for both repo1 (primary Postgres) and repo2 (insights Postgres). The weekly full backup schedule is unchanged.

Written by Cursor Bugbot for commit 8bd5bfc. This will update automatically on new commits. Configure here.

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix prepared a fix for the issue found in the latest run.

  • ✅ Fixed: Backup frequency silently reduced from hourly to daily
    • Restored the postgres-insights differential backup cron from daily (0 0 * * *) to hourly (0 */1 * * *) so only the backup type changed.

Create PR

Or push these changes by commenting:

@cursor push 473215fc03
Preview (473215fc03)
diff --git a/azure/application.tf b/azure/application.tf
--- a/azure/application.tf
+++ b/azure/application.tf
@@ -678,7 +678,7 @@
         container: " ${module.storage.crunchy_backup_name}"
       schedules:
         full: 30 4 * * *
-        differential: 0 0 * * * # Diff backup daily at midnight
+        differential: 0 */1 * * * # Diff backup hourly
     jobs:
       resources:
         requests:

This Bugbot Autofix run was free. To enable autofix for future PRs, go to the Cursor dashboard.

schedules:
full: 30 4 * * *
incremental: 0 */1 * * *
differential: 0 0 * * * # Diff backup daily at midnight
Copy link

Choose a reason for hiding this comment

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

Backup frequency silently reduced from hourly to daily

High Severity

The postgres-insights cluster's backup schedule was changed from hourly (0 */1 * * *) to daily (0 0 * * *) when switching from incremental to differential. The first cluster (postgres-data) correctly preserved its original daily schedule during the type change, but this second cluster's cron expression appears to have been copy-pasted from the first, silently reducing backup frequency from 24×/day to 1×/day. This significantly increases the recovery point objective (RPO) for the insights database.

Fix in Cursor Fix in Web

@NathanOkolita NathanOkolita merged commit 56b8679 into main Mar 13, 2026
20 checks passed
@NathanOkolita NathanOkolita deleted the dev-15000 branch March 13, 2026 16:49
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