lts-backup-cli 3.1.0__tar.gz → 3.2.0__tar.gz

{lts_backup_cli-3.1.0 → lts_backup_cli-3.2.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: lts-backup-cli
-Version: 3.1.0
+Version: 3.2.0
 Summary: LTS backup utility: a central backup server pulls Apache vhosts, MariaDB databases, folders and Docker hosts over SSH into local restic repositories, with discovery and scheduled rotation.
 Author-email: LTS <joaomartins@lts.pt>
 License: BSD-3-Clause
@@ -209,7 +209,7 @@ ltsbu discover lab.example.com --non-interactive \
 | `--repo-root`, `--mirror-root` | `/srv/restic`, `/srv/ltsbu/mirror` (on this backup server) |
 | `--db-user` / `--no-db-login` | password from `$LTSBU_DB_PASSWORD` / skip databases |
 | `--vhosts`, `--databases` | `all`, `none` or a comma-separated list; new server: all; edit: unchanged |
-| `--folders "/a /b"` | none |
+| `--folders "/a /b"` | `~/public_html` of the SSH user on a new server, if it exists and no vhost already covers it; otherwise none (`-` = none) |
 | `--docker all\|none`, `--docker-containers` | non-interactive: `none`; `all` = one repo per compose project plus `docker`; containers: `all`, `none` or a list (default: every running container with a `/var/lib/mysql` mount) |
 | `--every-hours N`, `--weekly-day DAY`, `--weekly-time HH:MM` | `2`, staggered night slot |
 | `--keep-hourly/-daily/-weekly/-monthly N` | `12 / 7 / 4 / 6` |
@@ -315,7 +315,7 @@ A server that already exists here needs `--force`, and then **keeps its local co
 | Option | Meaning |
 |---|---|
 | `--server LABEL`, `-s` | server to act on; optional when only one matches |
-| `--repo NAME`, `-r` | repo on that server; optional when only one matches |
+| `--repo NAME`, `-r`, `--label` | repo on that server; optional when only one matches |
 | `--dry-run` | print the commands that would change something instead of running them; read-only commands still run (a new server's `discover` logs in through a temporary copy of `~/.ssh/config` with the block it would add) |
 | `--verbose`, `-v` | log every external command (secrets are never on a command line) |
 | `--config FILE` | alternative config file (default `~/.config/ltsbu/config.toml`); put it before the command |

{lts_backup_cli-3.1.0 → lts_backup_cli-3.2.0}/README.md

@@ -174,7 +174,7 @@ ltsbu discover lab.example.com --non-interactive \
 | `--repo-root`, `--mirror-root` | `/srv/restic`, `/srv/ltsbu/mirror` (on this backup server) |
 | `--db-user` / `--no-db-login` | password from `$LTSBU_DB_PASSWORD` / skip databases |
 | `--vhosts`, `--databases` | `all`, `none` or a comma-separated list; new server: all; edit: unchanged |
-| `--folders "/a /b"` | none |
+| `--folders "/a /b"` | `~/public_html` of the SSH user on a new server, if it exists and no vhost already covers it; otherwise none (`-` = none) |
 | `--docker all\|none`, `--docker-containers` | non-interactive: `none`; `all` = one repo per compose project plus `docker`; containers: `all`, `none` or a list (default: every running container with a `/var/lib/mysql` mount) |
 | `--every-hours N`, `--weekly-day DAY`, `--weekly-time HH:MM` | `2`, staggered night slot |
 | `--keep-hourly/-daily/-weekly/-monthly N` | `12 / 7 / 4 / 6` |
@@ -280,7 +280,7 @@ A server that already exists here needs `--force`, and then **keeps its local co
 | Option | Meaning |
 |---|---|
 | `--server LABEL`, `-s` | server to act on; optional when only one matches |
-| `--repo NAME`, `-r` | repo on that server; optional when only one matches |
+| `--repo NAME`, `-r`, `--label` | repo on that server; optional when only one matches |
 | `--dry-run` | print the commands that would change something instead of running them; read-only commands still run (a new server's `discover` logs in through a temporary copy of `~/.ssh/config` with the block it would add) |
 | `--verbose`, `-v` | log every external command (secrets are never on a command line) |
 | `--config FILE` | alternative config file (default `~/.config/ltsbu/config.toml`); put it before the command |

{lts_backup_cli-3.1.0 → lts_backup_cli-3.2.0}/features/backup.feature

@@ -100,3 +100,16 @@ Feature: Back up a server's repos and rotate them
     When I run "ltsbu backup --server lab --repo docker --weekly"
     Then the exit code should be 0
     And the image "lts/electro-site:php7.2" should have been saved into restic
+
+  Scenario: --label is an alias of --repo
+    Given a configured server "lab" with repos
+      | repo   | kind   | what                |
+      | docker | docker | /srv/docker/volumes |
+      | proxy  | docker | /home/lts/proxy     |
+    When I run "ltsbu backup --server lab --label proxy"
+    Then the exit code should be 0
+    And rsync should have been called with "ltsbu-lab:/home/lts/proxy"
+    And rsync should not have been called with "ltsbu-lab:/srv/docker/volumes"
+    When I run "ltsbu --label docker backup --server lab"
+    Then the exit code should be 0
+    And rsync should have been called with "ltsbu-lab:/srv/docker/volumes"

{lts_backup_cli-3.1.0 → lts_backup_cli-3.2.0}/features/discover.feature

@@ -117,3 +117,10 @@ Feature: Discover a server and configure its backups
     When I run "ltsbu discover lab.example.com --non-interactive --ssh-user root --no-db-login --docker none"
     Then the exit code should be 0
     And server "lab" should have repos "blog-example-com, shop-example-com"
+
+  Scenario: A new server's public_html is proposed as an extra folder
+    Given the server's home is "/home/backup"
+    And the server has the paths "/home/backup/public_html"
+    When I run "ltsbu discover web1.example.com --non-interactive --ssh-user backup --no-db-login --vhosts none"
+    Then the exit code should be 0
+    And server "web1" should have repos "home-backup-public_html"

{lts_backup_cli-3.1.0 → lts_backup_cli-3.2.0}/src/lts_backup_cli/__init__.py

@@ -1,3 +1,3 @@
 """lts-backup-cli: restic + MariaDB backup utility."""
 
-__version__ = "3.1.0"
+__version__ = "3.2.0"

{lts_backup_cli-3.1.0 → lts_backup_cli-3.2.0}/src/lts_backup_cli/cli.py

@@ -123,7 +123,7 @@ def common(f: F) -> F:
     """--server, --repo, --dry-run, --verbose also accepted after the subcommand name."""
 
     @click.option("--server", "-s", metavar="LABEL", help="Server label.")
-    @click.option("--repo", "-r", metavar="NAME", help="Repo name on the server.")
+    @click.option("--repo", "-r", "--label", metavar="NAME", help="Repo name on the server.")
     @click.option(
         "--dry-run", is_flag=True, help="Print mutating commands instead of running them."
     )
@@ -158,7 +158,7 @@ def _confirm(st: State, yes: bool, message: str) -> None:
     help="Config file (default ~/.config/ltsbu/config.toml).",
 )
 @click.option("--server", "-s", metavar="LABEL", help="Server label.")
-@click.option("--repo", "-r", metavar="NAME", help="Repo name on the server.")
+@click.option("--repo", "-r", "--label", metavar="NAME", help="Repo name on the server.")
 @click.option("--dry-run", is_flag=True, help="Print mutating commands instead of running them.")
 @click.option("--verbose", "-v", is_flag=True, help="Show executed commands.")
 @click.version_option(__version__, prog_name="ltsbu")
@@ -231,7 +231,10 @@ def _run_doctor(st: State) -> None:
 @click.option("--no-db-login", is_flag=True, help="Do not set up a MariaDB login now.")
 @click.option("--vhosts", help="'all', 'none' or comma-separated repo names.")
 @click.option("--databases", help="'all', 'none' or comma-separated database names.")
-@click.option("--folders", help="Extra folders on the server, space separated.")
+@click.option(
+    "--folders",
+    help="Extra folders on the server, space separated; '-' for none [new server: ~/public_html if it exists].",
+)
 @click.option(
     "--docker", type=click.Choice(["all", "none"]), help="Back up the server's Docker host."
 )
@@ -311,7 +314,6 @@ def _discover_presets(non_interactive: bool, o: dict[str, Any]) -> dict[str, Any
     if non_interactive:
         p.setdefault("db_login", "db_user" in p)
         p.setdefault("docker", False)
-        p.setdefault("folders", "-")
         p.setdefault("customize", False)
         p["confirm_remove"] = o["remove"]
         p["apply"] = True

{lts_backup_cli-3.1.0 → lts_backup_cli-3.2.0}/src/lts_backup_cli/services.py

@@ -444,11 +444,14 @@ class DiscoverService:
         """docker.propose, keeping only paths that exist on the server (Portainer's compose
         dirs are container paths, already inside its data volume)."""
         split = docker.propose(found.containers, str(found.docker_root))
-        wanted = sorted({p for paths, _, _ in split.values() for p in paths})
-        out = runner.run(["sh", "-c", _EXISTING, "sh", *wanted], mutating=False).stdout
-        present = set(out.splitlines())
+        present = self.existing(runner, sorted({p for ps, _, _ in split.values() for p in ps}))
         return {n: ([p for p in ps if p in present], ex, dbs) for n, (ps, ex, dbs) in split.items()}
 
+    def existing(self, runner: Runner, paths: list[str]) -> set[str]:
+        """The `paths` that exist on the server (one constant script, paths as argv)."""
+        out = runner.run(["sh", "-c", _EXISTING, "sh", *paths], mutating=False).stdout
+        return set(out.splitlines())
+
     def databases(
         self, runner: Runner, server: c.ServerConfig
     ) -> tuple[list[str] | None, list[str]]:

{lts_backup_cli-3.1.0 → lts_backup_cli-3.2.0}/src/lts_backup_cli/wizard.py

@@ -251,6 +251,10 @@ def _db_login(
     return None
 
 
+def _overlaps(a: str, b: str) -> bool:
+    return a == b or a.startswith(b.rstrip("/") + "/") or b.startswith(a.rstrip("/") + "/")
+
+
 def _folder_name(path: str) -> str:
     return apache.repo_name_for(path.strip("/").replace("/", "-") or "root")
 
@@ -403,10 +407,14 @@ def run(
 
     # --- extra folders (one repo each) ---
     folder_paths = [p for r in existing.values() if r.kind == "folder" for p in r.paths]
+    default = " ".join(folder_paths) or "-"
+    if not old and found.home.startswith("/"):
+        pub = f"{found.home.rstrip('/')}/public_html"  # v1's default files path
+        covered = any(_overlaps(pub, h.document_root) for h in found.vhosts)
+        if not covered and pub in svc.existing(runner, [pub]):
+            default = pub
     raw = ask.text(
-        "folders",
-        "Extra folders on the server (space separated, empty for none):",
-        " ".join(folder_paths) or "-",
+        "folders", "Extra folders on the server (space separated, '-' for none):", default
     )
     for path in (p for p in raw.split() if p != "-"):
         path = v.remote_abs_path(path)

{lts_backup_cli-3.1.0 → lts_backup_cli-3.2.0}/tests/smoke/smoke.sh

@@ -20,6 +20,8 @@ done
 if ltsbu configure --help >/dev/null 2>&1; then fail "v1 'configure' command still present"; fi
 ltsbu restore --help | grep -q -- "--container" || fail "restore --container not wired"
 ltsbu restore --help | grep -q -- "--image" || fail "restore --image not wired"
+ltsbu backup --help | grep -q -- "--label" || fail "backup --label not wired"
+ltsbu --help | grep -q -- "--label" || fail "global --label not wired"
 ltsbu discover --help | grep -q -- "--docker" || fail "discover --docker not wired"
 if ltsbu discover --help | grep -q -- "--storage-host"; then fail "v2 --storage-host still present"; fi
 

{lts_backup_cli-3.1.0 → lts_backup_cli-3.2.0}/tests/unit/test_wizard.py

@@ -110,6 +110,38 @@ def test_select_subset_and_folders(server_runner: FakeRunner, tmp_path: Path) ->
     assert req.server.repos["srv-data"].kind == "folder"
 
 
+@pytest.mark.parametrize(
+    ("present", "vhost_root", "expected"),
+    [
+        ("/home/backup/public_html\n", "/var/www/shop", {"home-backup-public_html"}),
+        ("", "/var/www/shop", set()),  # no public_html on the server
+        ("/home/backup/public_html\n", "/home/backup/public_html", set()),  # already a vhost
+        ("/home/backup/public_html\n", "/home/backup/public_html/shop", set()),  # vhost inside
+    ],
+)
+def test_new_server_proposes_public_html_as_the_default_folder(
+    server_runner: FakeRunner, tmp_path: Path, present: str, vhost_root: str, expected: set[str]
+) -> None:
+    server_runner.rules = [r for r in server_runner.rules if r.stdout != APACHE]
+    server_runner.on(s._READ_APACHE, stdout=APACHE.replace("/var/www/shop", vhost_root))
+    server_runner.on(s._EXISTING, "/home/backup/public_html", stdout=present)
+    req = run(server_runner, c.Config(), tmp_path, fresh(vhosts=["shop-example-com"]))
+    folders = {n for n, r in req.server.repos.items() if r.kind == "folder"}
+    assert folders == expected
+    if expected:
+        assert req.server.repos["home-backup-public_html"].paths == ["/home/backup/public_html"]
+
+
+def test_editing_never_adds_public_html_by_itself(
+    server_runner: FakeRunner, tmp_path: Path
+) -> None:
+    server_runner.on(s._EXISTING, "/home/backup/public_html", stdout="/home/backup/public_html\n")
+    cfg = c.Config()
+    cfg.servers["web1"] = run(server_runner, cfg, tmp_path, fresh(folders="-")).server
+    again = run(server_runner, cfg, tmp_path, fresh()).server
+    assert not [r for r in again.repos.values() if r.kind == "folder"]
+
+
 def test_invalid_folder_is_rejected(server_runner: FakeRunner, tmp_path: Path) -> None:
     with pytest.raises(Exception, match="invalid remote path"):
         run(server_runner, c.Config(), tmp_path, fresh(folders="relative/dir"))