nix fmt

pkgs/clan-cli/README.md

@@ -70,7 +70,7 @@ pytest -n0 -s tests/test_secrets_cli.py::test_users
 ```
 
 ## Debugging functions
+
 Debugging functions can be found under `src/debug.py`
 quite interesting is the function repro_env_break() which drops you into a shell
 with the test environment loaded.
-

pkgs/clan-cli/clan_cli/debug.py

@@ -1,23 +1,27 @@
-from typing import Dict, Optional, Tuple, Callable, Any, Mapping, List
-from pathlib import Path
-import ipdb
+import logging
+import multiprocessing as mp
 import os
+import shlex
 import stat
 import subprocess
-from .dirs import find_git_repo_root
-import multiprocessing as mp
-from .types import FlakeName
-import logging
 import sys
-import shlex
-import time
+from pathlib import Path
+from typing import Any, Callable, Dict, List, Optional
+
+import ipdb
 
 log = logging.getLogger(__name__)
 
+
 def command_exec(cmd: List[str], work_dir: Path, env: Dict[str, str]) -> None:
     subprocess.run(cmd, check=True, env=env, cwd=work_dir.resolve())
 
-def repro_env_break(work_dir: Path, env: Optional[Dict[str, str]] = None, cmd: Optional[List[str]] = None) -> None:
+
+def repro_env_break(
+    work_dir: Path,
+    env: Optional[Dict[str, str]] = None,
+    cmd: Optional[List[str]] = None,
+) -> None:
     if env is None:
         env = os.environ.copy()
     else:
@@ -36,6 +40,7 @@ def repro_env_break(work_dir: Path, env: Optional[Dict[str, str]] = None, cmd: O
     finally:
         proc.terminate()
 
+
 def write_command(command: str, loc: Path) -> None:
     with open(loc, "w") as f:
         f.write("#!/usr/bin/env bash\n")
@@ -43,6 +48,7 @@ def write_command(command: str, loc:Path) -> None:
     st = os.stat(loc)
     os.chmod(loc, st.st_mode | stat.S_IEXEC)
 
+
 def spawn_process(func: Callable, **kwargs: Any) -> mp.Process:
     if mp.get_start_method(allow_none=True) is None:
         mp.set_start_method(method="spawn")
@@ -57,7 +63,7 @@ def dump_env(env: Dict[str, str], loc: Path) -> None:
     with open(loc, "w") as f:
         f.write("#!/usr/bin/env bash\n")
         for k, v in cenv.items():
-            if v.count('\n') > 0 or v.count("\"") > 0 or v.count("'") > 0:
+            if v.count("\n") > 0 or v.count('"') > 0 or v.count("'") > 0:
                 continue
             f.write(f"export {k}='{v}'\n")
     st = os.stat(loc)

pkgs/clan-cli/clan_cli/flakes/__init__.py

@@ -4,6 +4,7 @@ import argparse
 from .create import register_create_parser
 from .list import register_list_parser
 
+
 # takes a (sub)parser and configures it
 def register_parser(parser: argparse.ArgumentParser) -> None:
     subparser = parser.add_subparsers(

pkgs/clan-cli/clan_cli/secrets/sops_generate.py

@@ -1,3 +1,4 @@
+import logging
 import os
 import shlex
 import shutil
@@ -6,7 +7,6 @@ import sys
 from pathlib import Path
 from tempfile import TemporaryDirectory
 from typing import Any
-import logging
 
 from clan_cli.nix import nix_shell
 
@@ -20,6 +20,7 @@ from .sops import generate_private_key
 
 log = logging.getLogger(__name__)
 
+
 def generate_host_key(flake_name: FlakeName, machine_name: str) -> None:
     if has_machine(flake_name, machine_name):
         return
@@ -97,7 +98,9 @@ def generate_secrets_from_nix(
 ) -> None:
     generate_host_key(flake_name, machine_name)
     errors = {}
-    log.debug("Generating secrets for machine %s and flake %s", machine_name, flake_name)
+    log.debug(
+        "Generating secrets for machine %s and flake %s", machine_name, flake_name
+    )
     with TemporaryDirectory() as d:
         # if any of the secrets are missing, we regenerate all connected facts/secrets
         for secret_group, secret_options in secret_submodules.items():

pkgs/clan-cli/clan_cli/vms/create.py

@@ -2,20 +2,19 @@ import argparse
 import asyncio
 import json
 import os
+import re
 import shlex
 import sys
-import re
 from pathlib import Path
-from typing import Iterator, Dict
+from typing import Iterator
 from uuid import UUID
 
 from ..dirs import clan_flakes_dir, specific_flake_dir
+from ..errors import ClanError
 from ..nix import nix_build, nix_config, nix_eval, nix_shell
 from ..task_manager import BaseTask, Command, create_task
 from ..types import validate_path
 from .inspect import VmConfig, inspect_vm
-from ..errors import ClanError
-from ..debug import repro_env_break
 
 
 def is_path_or_url(s: str) -> str | None:
@@ -29,6 +28,7 @@ def is_path_or_url(s: str) -> str | None:
     else:
         return None
 
+
 class BuildVmTask(BaseTask):
     def __init__(self, uuid: UUID, vm: VmConfig) -> None:
         super().__init__(uuid, num_cmds=7)
@@ -105,7 +105,6 @@ class BuildVmTask(BaseTask):
             else:
                 self.log.warning("won't generate secrets for non local clan")
 
-
         cmd = next(cmds)
         cmd.run(
             [vm_config["uploadSecrets"], clan_name],

pkgs/clan-cli/clan_cli/webui/routers/machines.py

@@ -3,7 +3,7 @@ import logging
 from typing import Annotated
 
 from fastapi import APIRouter, Body
-from clan_cli.debug import repro_env_break
+
 from ...config.machine import (
     config_for_machine,
     schema_for_machine,

pkgs/clan-cli/clan_cli/webui/server.py

@@ -13,6 +13,7 @@ from typing import Iterator
 import uvicorn
 from pydantic import AnyUrl, IPvAnyAddress
 from pydantic.tools import parse_obj_as
+
 from clan_cli.errors import ClanError
 
 log = logging.getLogger(__name__)
@@ -25,8 +26,7 @@ def open_browser(base_url: AnyUrl, sub_url: str) -> None:
             break
         except OSError:
             time.sleep(i)
-    url = parse_obj_as(
-        AnyUrl,f"{base_url}/{sub_url.removeprefix('/')}")
+    url = parse_obj_as(AnyUrl, f"{base_url}/{sub_url.removeprefix('/')}")
     _open_browser(url)
 
 

pkgs/clan-cli/pyproject.toml

@@ -41,6 +41,10 @@ ignore_missing_imports = true
 module = "jsonschema.*"
 ignore_missing_imports = true
 
+[[tool.mypy.overrides]]
+module = "ipdb.*"
+ignore_missing_imports = true
+
 [[tool.mypy.overrides]]
 module = "pytest.*"
 ignore_missing_imports = true
@@ -52,7 +56,7 @@ ignore_missing_imports = true
 [tool.ruff]
 line-length = 88
 
-select = [ "E", "F", "I", "U", "N"]
+select = [ "E", "F", "I",  "N"]
 ignore = [ "E501" ]
 
 [tool.black]

pkgs/clan-cli/tests/fixtures_flakes.py

@@ -112,5 +112,4 @@ def test_flake_with_core_and_pass(
         temporary_home,
         FlakeName("test_flake_with_core_and_pass"),
         CLAN_CORE,
-
     )

pkgs/clan-cli/tests/test_config.py

@@ -5,11 +5,11 @@ from typing import Any, Optional
 
 import pytest
 from cli import Cli
+from fixtures_flakes import FlakeForTest
 
 from clan_cli import config
 from clan_cli.config import parsing
 from clan_cli.errors import ClanError
-from fixtures_flakes import FlakeForTest
 
 example_options = f"{Path(config.__file__).parent}/jsonschema/options.json"
 

pkgs/clan-cli/tests/test_flake_api.py

@@ -1,8 +1,8 @@
 import json
-from pathlib import Path
-from fixtures_flakes import FlakeForTest
+
 import pytest
 from api import TestClient
+from fixtures_flakes import FlakeForTest
 
 
 @pytest.mark.impure

pkgs/clan-cli/tests/test_import_sops_cli.py

@@ -1,10 +1,11 @@
 from pathlib import Path
 from typing import TYPE_CHECKING
-from fixtures_flakes import FlakeForTest
-from clan_cli.debug import repro_env_break
 
 import pytest
 from cli import Cli
+from fixtures_flakes import FlakeForTest
+
+from clan_cli.debug import repro_env_break
 
 if TYPE_CHECKING:
     from age_keys import KeyPair
@@ -20,7 +21,9 @@ def test_import_sops(
     cli = Cli()
 
     monkeypatch.setenv("SOPS_AGE_KEY", age_keys[1].privkey)
-    cli.run(["secrets", "machines", "add", "machine1", age_keys[0].pubkey, test_flake.name])
+    cli.run(
+        ["secrets", "machines", "add", "machine1", age_keys[0].pubkey, test_flake.name]
+    )
     cli.run(["secrets", "users", "add", "user1", age_keys[1].pubkey, test_flake.name])
     cli.run(["secrets", "users", "add", "user2", age_keys[2].pubkey, test_flake.name])
     cli.run(["secrets", "groups", "add-user", "group1", "user1", test_flake.name])
@@ -36,12 +39,10 @@ def test_import_sops(
         "--machine",
         "machine1",
         str(test_root.joinpath("data", "secrets.yaml")),
-            test_flake.name
+        test_flake.name,
     ]
     repro_env_break(work_dir=test_flake.path, cmd=cmd)
-    cli.run(
-        cmd
-    )
+    cli.run(cmd)
     capsys.readouterr()
     cli.run(["secrets", "users", "list", test_flake.name])
     users = sorted(capsys.readouterr().out.rstrip().split())

pkgs/clan-cli/tests/test_machines_api.py

@@ -1,14 +1,13 @@
-from pathlib import Path
-
 from api import TestClient
 from fixtures_flakes import FlakeForTest
-from clan_cli.debug import repro_env_break
+
 
 def test_machines(api: TestClient, test_flake: FlakeForTest) -> None:
     response = api.get(f"/api/{test_flake.name}/machines")
     assert response.status_code == 200
     assert response.json() == {"machines": []}
 
+    # TODO: Fails because the test_flake fixture needs to init a git repo, which it currently does not
     response = api.post(f"/api/{test_flake.name}/machines", json={"name": "test"})
     assert response.status_code == 201
     assert response.json() == {"machine": {"name": "test", "status": "unknown"}}

pkgs/clan-cli/tests/test_secrets_generate.py

@@ -21,7 +21,16 @@ def test_generate_secret(
     monkeypatch.chdir(test_flake_with_core.path)
     monkeypatch.setenv("SOPS_AGE_KEY", age_keys[0].privkey)
     cli = Cli()
-    cli.run(["secrets", "users", "add", "user1", age_keys[0].pubkey, test_flake_with_core.name])
+    cli.run(
+        [
+            "secrets",
+            "users",
+            "add",
+            "user1",
+            age_keys[0].pubkey,
+            test_flake_with_core.name,
+        ]
+    )
     cli.run(["secrets", "generate", "vm1", test_flake_with_core.name])
     has_secret(test_flake_with_core.name, "vm1-age.key")
     has_secret(test_flake_with_core.name, "vm1-zerotier-identity-secret")

pkgs/clan-cli/tests/test_secrets_upload.py

@@ -1,9 +1,9 @@
-from pathlib import Path
 from typing import TYPE_CHECKING
 
 import pytest
 from cli import Cli
 from fixtures_flakes import FlakeForTest
+
 from clan_cli.ssh import HostGroup
 
 if TYPE_CHECKING:
@@ -21,9 +21,27 @@ def test_secrets_upload(
     monkeypatch.setenv("SOPS_AGE_KEY", age_keys[0].privkey)
 
     cli = Cli()
-    cli.run(["secrets", "users", "add", "user1", age_keys[0].pubkey, test_flake_with_core.name])
+    cli.run(
+        [
+            "secrets",
+            "users",
+            "add",
+            "user1",
+            age_keys[0].pubkey,
+            test_flake_with_core.name,
+        ]
+    )
 
-    cli.run(["secrets", "machines", "add", "vm1", age_keys[1].pubkey, test_flake_with_core.name])
+    cli.run(
+        [
+            "secrets",
+            "machines",
+            "add",
+            "vm1",
+            age_keys[1].pubkey,
+            test_flake_with_core.name,
+        ]
+    )
     monkeypatch.setenv("SOPS_NIX_SECRET", age_keys[0].privkey)
     cli.run(["secrets", "set", "vm1-age.key", test_flake_with_core.name])
 

pkgs/clan-cli/tests/test_vms_api.py

@@ -1,9 +1,8 @@
-from pathlib import Path
-
 import pytest
 from api import TestClient
 from fixtures_flakes import FlakeForTest
 
+
 @pytest.mark.impure
 def test_inspect(api: TestClient, test_flake_with_core: FlakeForTest) -> None:
     response = api.post(

pkgs/clan-cli/tests/test_vms_api_create.py

@@ -9,7 +9,6 @@ from fixtures_flakes import FlakeForTest, create_flake
 from httpx import SyncByteStream
 from root import CLAN_CORE
 
-from clan_cli.debug import repro_env_break
 from clan_cli.types import FlakeName
 
 if TYPE_CHECKING:
@@ -18,8 +17,7 @@ if TYPE_CHECKING:
 
 @pytest.fixture
 def flake_with_vm_with_secrets(
-    monkeypatch: pytest.MonkeyPatch,
-    temporary_home: Path
+    monkeypatch: pytest.MonkeyPatch, temporary_home: Path
 ) -> Iterator[FlakeForTest]:
     yield from create_flake(
         monkeypatch,
@@ -44,8 +42,6 @@ def remote_flake_with_vm_without_secrets(
     )
 
 
-
-
 def generic_create_vm_test(api: TestClient, flake: Path, vm: str) -> None:
     print(f"flake_url: {flake} ")
     response = api.post(
@@ -94,7 +90,14 @@ def test_create_local(
 ) -> None:
     monkeypatch.setenv("SOPS_AGE_KEY", age_keys[0].privkey)
     cli = Cli()
-    cmd = ["secrets", "users", "add", "user1", age_keys[0].pubkey, flake_with_vm_with_secrets.name]
+    cmd = [
+        "secrets",
+        "users",
+        "add",
+        "user1",
+        age_keys[0].pubkey,
+        flake_with_vm_with_secrets.name,
+    ]
     cli.run(cmd)
 
     generic_create_vm_test(api, flake_with_vm_with_secrets.path, "vm_with_secrets")

pkgs/clan-cli/tests/test_vms_cli.py

@@ -1,9 +1,9 @@
 import os
-from pathlib import Path
 from typing import TYPE_CHECKING
-from fixtures_flakes import FlakeForTest
+
 import pytest
 from cli import Cli
+from fixtures_flakes import FlakeForTest
 
 if TYPE_CHECKING:
     from age_keys import KeyPair
@@ -12,7 +12,9 @@ no_kvm = not os.path.exists("/dev/kvm")
 
 
 @pytest.mark.impure
-def test_inspect(test_flake_with_core: FlakeForTest, capsys: pytest.CaptureFixture) -> None:
+def test_inspect(
+    test_flake_with_core: FlakeForTest, capsys: pytest.CaptureFixture
+) -> None:
     cli = Cli()
     cli.run(["vms", "inspect", "vm1", test_flake_with_core.name])
     out = capsys.readouterr()  # empty the buffer
@@ -29,5 +31,14 @@ def test_create(
     monkeypatch.chdir(test_flake_with_core.path)
     monkeypatch.setenv("SOPS_AGE_KEY", age_keys[0].privkey)
     cli = Cli()
-    cli.run(["secrets", "users", "add", "user1", age_keys[0].pubkey, test_flake_with_core.name])
+    cli.run(
+        [
+            "secrets",
+            "users",
+            "add",
+            "user1",
+            age_keys[0].pubkey,
+            test_flake_with_core.name,
+        ]
+    )
     cli.run(["vms", "create", "vm1", test_flake_with_core.name])