From 60ff14d6b72c817387a0bc150b712d6ebfee4c84 Mon Sep 17 00:00:00 2001 From: Michael Hoang Date: Fri, 25 Apr 2025 16:47:21 +1000 Subject: [PATCH] cli: fix restoring backups There was a bug in `select` that made it output attrsets instead of lists so we fix the broken refactor done in 5ac629f549f7c40568643e12ff11d898f8af5521. --- pkgs/clan-cli/clan_cli/backups/restore.py | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/pkgs/clan-cli/clan_cli/backups/restore.py b/pkgs/clan-cli/clan_cli/backups/restore.py index 49902dc4b..d9746b32e 100644 --- a/pkgs/clan-cli/clan_cli/backups/restore.py +++ b/pkgs/clan-cli/clan_cli/backups/restore.py @@ -18,10 +18,7 @@ def restore_service(machine: Machine, name: str, provider: str, service: str) -> msg = f"Service {service} not found in configuration. Available services are: {', '.join(backup_folders.keys())}" raise ClanError(msg) - folders = backup_folders[service]["folders"].values() - assert all(isinstance(f, str) for f in folders), ( - f"folders must be a list of strings instead of {folders}" - ) + folders = backup_folders[service]["folders"] env = {} env["NAME"] = name # FIXME: If we have too many folder this might overflow the stack.