launchpad-reviewers team mailing list archive

[Barry Price <mp+478639@xxxxxxxxxxxxxxxxxx>]

Barry Price has proposed merging ~barryprice/launchpad/+git/launchpad:master into launchpad:master.

Commit message:
launchpad-postgresql-extras/push-backups: Delete older files excluded from the sync from the staging target

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~barryprice/launchpad/+git/launchpad/+merge/478639
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~barryprice/launchpad/+git/launchpad:master into launchpad:master.

diff --git a/charm/launchpad-postgresql-extras/files/push-backups b/charm/launchpad-postgresql-extras/files/push-backups
index 9a781c0..6545698 100755
--- a/charm/launchpad-postgresql-extras/files/push-backups
+++ b/charm/launchpad-postgresql-extras/files/push-backups
@@ -71,7 +71,7 @@ def main():
         command.append(f"--rsh=ssh -i {shlex.quote(args.ssh_key)}")
     else:
         command.append("--rsh=ssh")
-    command.extend(["-a", "--delete-after"])
+    command.extend(["-a", "--delete-after", "--delete-excluded"])
     for database in args.databases:
         for path in find_latest_dumps(args.backups_path, database, args.count):
             command.extend(["--include", path.name])