diff --git a/checks/update/flake-module.nix b/checks/update/flake-module.nix index c2f8af6f0..49595ecf3 100644 --- a/checks/update/flake-module.nix +++ b/checks/update/flake-module.nix @@ -236,7 +236,7 @@ "--debug", "--flake", "/flake", "--host-key-check", "none", - "--fetch-local", # Use local store instead of fetching from network + "--upload-inputs", # Use local store instead of fetching from network "--build-host", "local", "test-update-machine", "--target-host", f"root@localhost", @@ -247,12 +247,12 @@ ############## - print("TEST: update with --fetch-local") + print("TEST: update with --target-host") with open(machine_config_path, "w") as f: f.write(""" { - environment.etc."update-fetch-local-successful".text = "ok"; + environment.etc."target-host-update-successful".text = "ok"; } """) @@ -264,17 +264,17 @@ "--debug", "--flake", flake_dir, "--host-key-check", "none", - "--fetch-local", # Use local store instead of fetching from network + "--upload-inputs", # Use local store instead of fetching from network "test-update-machine", "--target-host", f"root@192.168.1.1:{ssh_conn.host_port}", ], check=True) # Verify the update was successful - machine.succeed("test -f /etc/update-fetch-local-successful") + machine.succeed("test -f /etc/target-host-update-successful") ############## - print("TEST: update with --build-host 192.168.1.1") + print("TEST: update with --build-host") # Update configuration again with open(machine_config_path, "w") as f: f.write(""" @@ -291,7 +291,7 @@ "--debug", "--flake", flake_dir, "--host-key-check", "none", - "--fetch-local", # Use local store instead of fetching from network + "--upload-inputs", # Use local store instead of fetching from network "--build-host", f"root@192.168.1.1:{ssh_conn.host_port}", "test-update-machine", "--target-host", f"root@192.168.1.1:{ssh_conn.host_port}", diff --git a/pkgs/clan-cli/clan_cli/machines/update.py b/pkgs/clan-cli/clan_cli/machines/update.py index da19c0932..d9ef8e58e 100644 --- a/pkgs/clan-cli/clan_cli/machines/update.py +++ b/pkgs/clan-cli/clan_cli/machines/update.py @@ -162,7 +162,7 @@ def update_command(args: argparse.Namespace) -> None: machine=machine, target_host=target_host, build_host=build_host, - force_fetch_local=args.fetch_local, + upload_inputs=args.upload_inputs, ) runtime.join_all() runtime.check_all() @@ -211,10 +211,10 @@ def register_update_parser(parser: argparse.ArgumentParser) -> None: ), ) parser.add_argument( - "--fetch-local", + "--upload-inputs", action="store_true", help=( - "Prefetch flake inputs locally, then upload them to the build-host.\n" + "Upload all flake inputs from the local machine instead of the build host/target host.\n" "This is useful if downloading the inputs requires authentication " "which is only available to the local machine" ), diff --git a/pkgs/clan-cli/clan_lib/machines/update.py b/pkgs/clan-cli/clan_lib/machines/update.py index fe5919e8e..de447e43f 100644 --- a/pkgs/clan-cli/clan_lib/machines/update.py +++ b/pkgs/clan-cli/clan_lib/machines/update.py @@ -37,7 +37,7 @@ def is_local_input(node: dict[str, dict[str, str]]) -> bool: return local -def upload_sources(machine: Machine, ssh: Host, force_fetch_local: bool) -> str: +def upload_sources(machine: Machine, ssh: Host, upload_inputs: bool) -> str: env = ssh.nix_ssh_env(os.environ.copy()) flake_url = ( @@ -53,7 +53,7 @@ def upload_sources(machine: Machine, ssh: Host, force_fetch_local: bool) -> str: remote_program_params = "" # MacOS doesn't come with a proper login shell for ssh and therefore doesn't have nix in $PATH as it doesn't source /etc/profile - if not has_path_inputs and not force_fetch_local: + if not has_path_inputs and not upload_inputs: # Just copy the flake to the remote machine, we can substitute other inputs there. path = flake_data["path"] if machine._class_ == "darwin": @@ -120,14 +120,14 @@ def run_machine_update( machine: Machine, target_host: Host, build_host: Host | None, - force_fetch_local: bool = False, + upload_inputs: bool = False, ) -> None: """Update an existing machine using nixos-rebuild or darwin-rebuild. Args: machine: The Machine instance to deploy. target_host: Remote object representing the target host for deployment. build_host: Optional Remote object representing the build host. - force_fetch_local: Whether to fetch flake inputs locally before uploading. + upload_inputs: Whether to upload flake inputs from the local. Raises: ClanError: If the machine is not found in the inventory or if there are issues with generating facts or variables. @@ -153,7 +153,7 @@ def run_machine_update( upload_secret_vars(machine, target_host_root) # Upload the flake's source to the build host. - path = upload_sources(machine, build_host, force_fetch_local) + path = upload_sources(machine, build_host, upload_inputs) nix_options = machine.flake.nix_options if machine.flake.nix_options else [] @@ -215,11 +215,11 @@ def run_machine_update( # retry nixos-rebuild switch if the first attempt failed if ret.returncode != 0: - # Hint user to --fetch-local on issues with flake inputs + # Hint user to --upload-inputs on issues with flake inputs if "… while fetching the input" in ret.stderr: msg = ( "Detected potential issue when fetching flake inputs on remote." - "\nTry running the update with --fetch-local to prefetch inputs " + "\nTry running the update with --update-inputs to prefetch inputs " "locally and upload them instead." ) raise ClanError(msg)