Merge ~cjwatson/launchpad:charm-postgresql-extras-push-backups-primary-only into launchpad:master

[Colin Watson]

charm: Only run push-backups on primary database units

We don't want to be pushing backups from more than one unit.

Unmerged commits

513b1bc... by Colin Watson on 2023-11-26

charm: Only run push-backups on primary database units

We don't want to be pushing backups from more than one unit.

	docs:0 (build)
	lint:0 (build)
	mypy:0 (build)

1 → 3 of 3 results

First • Previous • Next • Last

diff --git a/charm/launchpad-postgresql-extras/files/push-backups b/charm/launchpad-postgresql-extras/files/push-backups
index 8a48574..9e2989d 100755
--- a/charm/launchpad-postgresql-extras/files/push-backups
+++ b/charm/launchpad-postgresql-extras/files/push-backups
@@ -9,6 +9,18 @@ from argparse import ArgumentParser
 from pathlib import Path
 
 
+def is_primary():
+    return (
+        subprocess.run(
+            ["psql", "-At", "-c", "SELECT pg_is_in_recovery();"],
+            capture_output=True,
+            check=True,
+            text=True,
+        ).stdout.strip()
+        == "f"
+    )
+
+
 def find_latest_dumps(backups_path, database, count):
     yield from sorted(backups_path.glob(f"{glob.escape(database)}.*.dump"))[
         -count:
@@ -49,6 +61,9 @@ def main():
     parser.add_argument("databases", metavar="database", nargs="+")
     args = parser.parse_args()
 
+    if not is_primary():
+        return
+
     command = ["rsync"]
     if args.bwlimit is not None:
         command.append(f"--bwlimit={args.bwlimit}")