gpustack-runtime 0.1.39__py3-none-any.whl → 0.1.39.post2__py3-none-any.whl

gpustack_runtime/deployer/podman.py

@@ -1,12 +1,14 @@
 from __future__ import annotations
 
 import contextlib
+import io
 import json
 import logging
 import operator
 import os
 import socket
 import sys
+import tarfile
 from dataclasses import dataclass, field
 from functools import lru_cache, reduce
 from math import ceil
@@ -26,7 +28,7 @@ from podman.domain.containers_create import CreateMixin
 from tqdm import tqdm
 
 from .. import envs
-from ..logging import debug_log_exception
+from ..logging import debug_log_exception, debug_log_warning
 from .__patches__ import patch_render_payload
 from .__types__ import (
     Container,
@@ -35,7 +37,7 @@ from .__types__ import (
     ContainerMountModeEnum,
     ContainerProfileEnum,
     ContainerRestartPolicyEnum,
-    Deployer,
+    EndoscopicDeployer,
     OperationError,
     UnsupportedError,
     WorkloadExecStream,
@@ -47,7 +49,13 @@ from .__types__ import (
     WorkloadStatusOperation,
     WorkloadStatusStateEnum,
 )
-from .__utils__ import _MiB, bytes_to_human_readable, replace_image_with, safe_json
+from .__utils__ import (
+    _MiB,
+    bytes_to_human_readable,
+    replace_image_with,
+    safe_json,
+    sensitive_env_var,
+)
 
 if TYPE_CHECKING:
     from collections.abc import Callable, Generator
@@ -142,7 +150,7 @@ class PodmanWorkloadPlan(WorkloadPlan):
         super().validate_and_default()
 
         # Adjust default image namespace if needed.
-        if namespace := envs.GPUSTACK_RUNTIME_DEPLOY_DEFAULT_IMAGE_NAMESPACE:
+        if namespace := envs.GPUSTACK_RUNTIME_DEPLOY_DEFAULT_CONTAINER_NAMESPACE:
             self.pause_image = replace_image_with(
                 image=self.pause_image,
                 namespace=namespace,
@@ -297,7 +305,7 @@ Name of the Podman deployer.
 """
 
 
-class PodmanDeployer(Deployer):
+class PodmanDeployer(EndoscopicDeployer):
     """
     Deployer implementation for Podman containers.
     """
@@ -306,10 +314,6 @@ class PodmanDeployer(Deployer):
     """
     Client for interacting with the Podman daemon.
     """
-    _container_ephemeral_files_dir: Path | None = None
-    """
-    Directory for ephemeral files inside containers, internal use only.
-    """
     _mutate_create_options: Callable[[dict[str, Any]], dict[str, Any]] | None = None
     """
     Function to handle mirrored deployment, internal use only.
@@ -386,48 +390,6 @@ class PodmanDeployer(Deployer):
 
         return wrapper
 
-    @staticmethod
-    def _create_ephemeral_files(
-        workload: PodmanWorkloadPlan,
-    ) -> dict[tuple[int, str], str]:
-        """
-        Create ephemeral files as local file for the workload.
-
-        Returns:
-            A mapping from (container index, configured path) to actual filename.
-
-        Raises:
-            OperationError:
-                If the ephemeral files fail to create.
-
-        """
-        # Map (container index, configured path) to actual filename.
-        ephemeral_filename_mapping: dict[tuple[int, str], str] = {}
-        ephemeral_files: list[tuple[str, str, int]] = []
-        for ci, c in enumerate(workload.containers):
-            for fi, f in enumerate(c.files or []):
-                if f.content is not None:
-                    fn = f"{workload.name}-{ci}-{fi}"
-                    ephemeral_filename_mapping[(ci, f.path)] = fn
-                    ephemeral_files.append((fn, f.content, f.mode))
-        if not ephemeral_filename_mapping:
-            return ephemeral_filename_mapping
-
-        # Create ephemeral files directory if not exists.
-        try:
-            for fn, fc, fm in ephemeral_files:
-                fp = envs.GPUSTACK_RUNTIME_PODMAN_EPHEMERAL_FILES_DIR.joinpath(fn)
-                with fp.open("w", encoding="utf-8") as f:
-                    f.write(fc)
-                    f.flush()
-                fp.chmod(fm)
-                logger.debug("Created local file %s with mode %s", fp, oct(fm))
-        except OSError as e:
-            msg = "Failed to create ephemeral files"
-            raise OperationError(msg) from e
-
-        return ephemeral_filename_mapping
-
     def _create_ephemeral_volumes(self, workload: PodmanWorkloadPlan) -> dict[str, str]:
         """
         Create ephemeral volumes for the workload.
@@ -473,12 +435,12 @@ class PodmanDeployer(Deployer):
             tag = tag or "latest"
             auth_config = None
             if (
-                envs.GPUSTACK_RUNTIME_DEPLOY_DEFAULT_IMAGE_REGISTRY_USERNAME
-                and envs.GPUSTACK_RUNTIME_DEPLOY_DEFAULT_IMAGE_REGISTRY_PASSWORD
+                envs.GPUSTACK_RUNTIME_DEPLOY_DEFAULT_CONTAINER_REGISTRY_USERNAME
+                and envs.GPUSTACK_RUNTIME_DEPLOY_DEFAULT_CONTAINER_REGISTRY_PASSWORD
             ):
                 auth_config = {
-                    "username": envs.GPUSTACK_RUNTIME_DEPLOY_DEFAULT_IMAGE_REGISTRY_USERNAME,
-                    "password": envs.GPUSTACK_RUNTIME_DEPLOY_DEFAULT_IMAGE_REGISTRY_PASSWORD,
+                    "username": envs.GPUSTACK_RUNTIME_DEPLOY_DEFAULT_CONTAINER_REGISTRY_USERNAME,
+                    "password": envs.GPUSTACK_RUNTIME_DEPLOY_DEFAULT_CONTAINER_REGISTRY_PASSWORD,
                 }
 
             logs = self._client.api.pull(
@@ -715,12 +677,10 @@ class PodmanDeployer(Deployer):
         else:
             return d_container
 
+    @staticmethod
     def _append_container_mounts(
-        self,
         create_options: dict[str, Any],
         c: Container,
-        ci: int,
-        ephemeral_filename_mapping: dict[tuple[int, str] : str],
         ephemeral_volume_name_mapping: dict[str, str],
     ):
         """
@@ -736,17 +696,7 @@ class PodmanDeployer(Deployer):
                     "target": "",
                 }
 
-                if f.content is not None:
-                    # Ephemeral file, use from local ephemeral files directory.
-                    if (ci, f.path) not in ephemeral_filename_mapping:
-                        continue
-                    fn = ephemeral_filename_mapping[(ci, f.path)]
-                    path = str(
-                        self._container_ephemeral_files_dir.joinpath(fn),
-                    )
-                    binding["source"] = path
-                    binding["target"] = f"/{f.path.lstrip('/')}"
-                elif f.path:
+                if f.content is None and f.path:
                     # Host file, bind directly.
                     binding["source"] = f.path
                     binding["target"] = f.path
@@ -858,10 +808,39 @@ class PodmanDeployer(Deployer):
 
         return healthcheck
 
+    @staticmethod
+    def _upload_ephemeral_files(
+        c: Container,
+        container: podman.domain.containers.Container,
+    ):
+        if not c.files:
+            return
+
+        f_tar = io.BytesIO()
+        with tarfile.open(fileobj=f_tar, mode="w") as tar:
+            for f in c.files:
+                if f.content is None or not f.path:
+                    continue
+                fc_bytes = f.content.encode("utf-8")
+                info = tarfile.TarInfo(name=f.path.lstrip("/"))
+                info.size = len(fc_bytes)
+                info.mode = f.mode
+                tar.addfile(tarinfo=info, fileobj=io.BytesIO(fc_bytes))
+        if f_tar.getbuffer().nbytes == 0:
+            return
+
+        f_tar.seek(0)
+        uploaded = container.put_archive(
+            path="/",
+            data=f_tar.getvalue(),
+        )
+        if not uploaded:
+            msg = f"Failed to upload ephemeral files to container {container.name}"
+            raise OperationError(msg)
+
     def _create_containers(
         self,
         workload: PodmanWorkloadPlan,
-        ephemeral_filename_mapping: dict[tuple[int, str] : str],
         ephemeral_volume_name_mapping: dict[str, str],
         pause_container: podman.domain.containers.Container,
     ) -> (
@@ -1077,8 +1056,6 @@ class PodmanDeployer(Deployer):
             self._append_container_mounts(
                 create_options,
                 c,
-                ci,
-                ephemeral_filename_mapping,
                 ephemeral_volume_name_mapping,
             )
 
@@ -1125,6 +1102,10 @@ class PodmanDeployer(Deployer):
                         detach=detach,
                         **create_options,
                     )
+
+                # Upload ephemeral files into the container.
+                self._upload_ephemeral_files(c, d_container)
+
             except podman.errors.APIError as e:
                 msg = f"Failed to create container {container_name}{_detail_api_call_error(e)}"
                 raise OperationError(msg) from e
@@ -1174,43 +1155,30 @@ class PodmanDeployer(Deployer):
     def __init__(self):
         super().__init__(_NAME)
         self._client = self._get_client()
-        self._container_ephemeral_files_dir = (
-            envs.GPUSTACK_RUNTIME_PODMAN_EPHEMERAL_FILES_DIR
-        )
 
-    def _prepare_create(self):
+    def _prepare_mirrored_deployment(self):
         """
-        Prepare for creation.
+        Prepare for mirrored deployment.
 
         """
         # Prepare mirrored deployment if enabled.
         if self._mutate_create_options:
             return
         self._mutate_create_options = lambda o: o
-        if not envs.GPUSTACK_RUNTIME_DEPLOY_MIRRORED_DEPLOYMENT:
-            logger.debug("Mirrored deployment disabled")
-            return
 
         # Retrieve self-container info.
-        ## - Get Container name, default to hostname if not set.
-        self_container_id = envs.GPUSTACK_RUNTIME_DEPLOY_MIRRORED_NAME
-        if not self_container_id:
-            self_container_id = socket.gethostname()
-            logger.warning(
-                "Mirrored deployment enabled, but no Container name set, using hostname(%s) instead",
-                self_container_id,
-            )
         try:
-            self_container = self._find_self_container(self_container_id)
+            self_container = self._find_self_container()
+            if not self_container:
+                return
             logger.info(
                 "Mirrored deployment enabled, using self Container %s for options mirroring",
-                self_container.id[:12],
+                self_container.short_id,
             )
             self_image = self_container.image
         except podman.errors.APIError:
             logger.exception(
-                "Mirrored deployment enabled, but failed to get self Container %s, skipping",
-                self_container_id,
+                "Mirrored deployment enabled, but failed to get self Container, skipping",
             )
             return
 
@@ -1370,36 +1338,10 @@ class PodmanDeployer(Deployer):
 
         self._mutate_create_options = mutate_create_options
 
-        # Extract ephemeral files dir mutation if any.
-        if mirrored_mounts:
-            e_target = str(envs.GPUSTACK_RUNTIME_PODMAN_EPHEMERAL_FILES_DIR)
-            b_source = ""
-            b_target = ""
-            for m in mirrored_mounts:
-                c_target = m.get("Destination", "///")
-                if (
-                    e_target == c_target or e_target.startswith(f"{c_target}/")
-                ) and len(c_target) >= len(b_target):
-                    b_source = m.get("Source")
-                    b_target = c_target
-            if b_source:
-                result = Path(b_source)
-                if e_target != b_target:
-                    b_subpath = e_target.removeprefix(b_target)
-                    result = result.joinpath(b_subpath.lstrip("/"))
-                self._container_ephemeral_files_dir = result
-
-    def _find_self_container(
-        self,
-        self_container_id: str,
-    ) -> podman.domain.containers.Container:
+    def _find_self_container(self) -> podman.domain.containers.Container | None:
         """
         Find the current container if running inside a Podman container.
 
-        Args:
-            self_container_id:
-                The container name or ID to find.
-
         Returns:
             The Podman container if found, None otherwise.
 
@@ -1407,38 +1349,54 @@ class PodmanDeployer(Deployer):
             If failed to find itself.
 
         """
-        if envs.GPUSTACK_RUNTIME_DEPLOY_MIRRORED_NAME:
-            # Directly get container by name or ID.
-            return self._client.containers.get(self_container_id)
-
-        # Find containers that matches the hostname.
-        containers: list[podman.domain.containers.Container] = []
-        for c in self._client.containers.list(compatible=True):
-            # Ignore workload containers with host network enabled.
-            if _LABEL_WORKLOAD in c.labels:
-                continue
-            # Ignore containers that do not match the hostname.
-            if c.attrs["Config"].get("Hostname", "") != self_container_id:
-                continue
-            # Ignore containers that do not match the filter labels.
-            if envs.GPUSTACK_RUNTIME_PODMAN_MIRRORED_NAME_FILTER_LABELS and any(
-                c.labels.get(k) != v
-                for k, v in envs.GPUSTACK_RUNTIME_PODMAN_MIRRORED_NAME_FILTER_LABELS.items()
-            ):
-                continue
-            containers.append(c)
+        if not envs.GPUSTACK_RUNTIME_DEPLOY_MIRRORED_DEPLOYMENT:
+            logger.debug("Mirrored deployment disabled")
+            return None
 
-        # Validate found containers.
-        if len(containers) != 1:
-            msg = (
-                f"Found multiple Containers with the same hostname {self_container_id}, "
-                if len(containers) > 1
-                else f"Not found Container with hostname {self_container_id}, "
-                "please use `--env GPUSTACK_RUNTIME_DEPLOY_MIRRORED_NAME=...` to specify the exact container name"
+        # Get container ID or hostname.
+        self_container_id = envs.GPUSTACK_RUNTIME_DEPLOY_MIRRORED_NAME
+        if not self_container_id:
+            self_container_id = socket.gethostname()
+            debug_log_warning(
+                logger,
+                "Mirrored deployment enabled, but no Container name set, using hostname(%s) instead",
+                self_container_id,
             )
-            raise podman.errors.NotFound(msg)
 
-        return containers[0]
+        if envs.GPUSTACK_RUNTIME_DEPLOY_MIRRORED_NAME:
+            # Directly get container.
+            self_container = self._client.containers.get(self_container_id)
+        else:
+            # Find containers that matches the hostname.
+            containers: list[podman.domain.containers.Container] = []
+            for c in self._client.containers.list(compatible=True):
+                # Ignore workload containers with host network enabled.
+                if _LABEL_WORKLOAD in c.labels:
+                    continue
+                # Ignore containers that do not match the hostname.
+                if c.attrs["Config"].get("Hostname", "") != self_container_id:
+                    continue
+                # Ignore containers that do not match the filter labels.
+                if envs.GPUSTACK_RUNTIME_PODMAN_MIRRORED_NAME_FILTER_LABELS and any(
+                    c.labels.get(k) != v
+                    for k, v in envs.GPUSTACK_RUNTIME_PODMAN_MIRRORED_NAME_FILTER_LABELS.items()
+                ):
+                    continue
+                containers.append(c)
+
+            # Validate found containers.
+            if len(containers) != 1:
+                msg = (
+                    f"Found multiple Containers with the same hostname {self_container_id}, "
+                    if len(containers) > 1
+                    else f"Not found Container with hostname {self_container_id}, "
+                    "please use `--env GPUSTACK_RUNTIME_DEPLOY_MIRRORED_NAME=...` to specify the exact Container name"
+                )
+                raise podman.errors.NotFound(msg)
+
+            self_container = containers[0]
+
+        return self_container
 
     @_supported
     def _create(self, workload: WorkloadPlan):
@@ -1464,7 +1422,7 @@ class PodmanDeployer(Deployer):
             msg = f"Invalid workload type: {type(workload)}"
             raise TypeError(msg)
 
-        self._prepare_create()
+        self._prepare_mirrored_deployment()
 
         if isinstance(workload, WorkloadPlan):
             workload = PodmanWorkloadPlan(**workload.__dict__)
@@ -1472,12 +1430,6 @@ class PodmanDeployer(Deployer):
         if logger.isEnabledFor(logging.DEBUG):
             logger.debug("Creating workload:\n%s", workload.to_yaml())
 
-        # Create ephemeral file if needed,
-        # (container index, configured path): <actual filename>
-        ephemeral_filename_mapping: dict[tuple[int, str] : str] = (
-            self._create_ephemeral_files(workload)
-        )
-
         # Create ephemeral volumes if needed,
         # <configured volume name>: <actual volume name>
         ephemeral_volume_name_mapping: dict[str, str] = self._create_ephemeral_volumes(
@@ -1490,7 +1442,6 @@ class PodmanDeployer(Deployer):
         # Create init/run containers.
         init_containers, run_containers = self._create_containers(
             workload,
-            ephemeral_filename_mapping,
             ephemeral_volume_name_mapping,
             pause_container,
         )
@@ -1631,17 +1582,6 @@ class PodmanDeployer(Deployer):
             msg = f"Failed to delete volumes for workload {name}{_detail_api_call_error(e)}"
             raise OperationError(msg) from e
 
-        # Remove all ephemeral files for the workload.
-        try:
-            for fp in envs.GPUSTACK_RUNTIME_PODMAN_EPHEMERAL_FILES_DIR.glob(
-                f"{name}-*",
-            ):
-                if fp.is_file():
-                    fp.unlink(missing_ok=True)
-        except OSError as e:
-            msg = f"Failed to delete ephemeral files for workload {name}"
-            raise OperationError(msg) from e
-
         return workload
 
     @_supported
@@ -1796,6 +1736,11 @@ class PodmanDeployer(Deployer):
             msg = f"Failed to fetch logs for container {container.name} of workload {name}{_detail_api_call_error(e)}"
             raise OperationError(msg) from e
         else:
+            if not follow:
+                result = bytearray()
+                for chunk in output:
+                    result.extend(chunk)
+                return result.decode("utf-8")
             return output
 
     @_supported
@@ -1880,6 +1825,216 @@ class PodmanDeployer(Deployer):
                 return output
             return PodmanWorkloadExecStream(output)
 
+    @_supported
+    def _inspect(
+        self,
+        name: WorkloadName,
+        namespace: WorkloadNamespace | None = None,
+    ) -> str | None:
+        """
+        Inspect a Podman workload.
+
+        Args:
+            name:
+                The name of the workload.
+            namespace:
+                The namespace of the workload.
+
+        Returns:
+            The inspection result as a JSON string. None if not found.
+
+        Raises:
+            UnsupportedError:
+                If Podman is not supported in the current environment.
+            OperationError:
+                If the Podman workload fails to inspect.
+
+        """
+        workload = self._get(name=name, namespace=namespace)
+        if not workload:
+            return None
+
+        d_containers = getattr(workload, "_d_containers", [])
+        if not d_containers:
+            return None
+
+        result = []
+        for c in d_containers:
+            c_attrs = c.attrs
+            # Mask sensitive environment variables
+            if "Env" in c_attrs["Config"]:
+                for i, env in enumerate(c_attrs["Config"]["Env"] or []):
+                    env_name, _ = env.split("=", maxsplit=1)
+                    if sensitive_env_var(env_name):
+                        c_attrs["Config"]["Env"][i] = f"{env_name}=******"
+            result.append(c_attrs)
+        return safe_json(result, indent=2)
+
+    def _find_self_container_for_endoscopy(self) -> podman.domain.containers.Container:
+        """
+        Find the self container for endoscopy.
+        Only works in mirrored deployment mode.
+
+        Returns:
+            The self container object.
+
+        Raises:
+            UnsupportedError:
+                If endoscopy is not supported in the current environment.
+
+        """
+        try:
+            self_container = self._find_self_container()
+        except podman.errors.APIError as e:
+            msg = "Endoscopy is not supported in the current environment: Mirrored deployment enabled, but failed to get self Container"
+            raise UnsupportedError(msg) from e
+        except Exception as e:
+            msg = "Endoscopy is not supported in the current environment: Failed to get self Container"
+            raise UnsupportedError(msg) from e
+
+        if not self_container:
+            msg = "Endoscopy is not supported in the current environment: Mirrored deployment disabled"
+            raise UnsupportedError(msg)
+        return self_container
+
+    def _endoscopic_logs(
+        self,
+        timestamps: bool = False,
+        tail: int | None = None,
+        since: int | None = None,
+        follow: bool = False,
+    ) -> Generator[bytes | str, None, None] | bytes | str:
+        """
+        Get the logs of the deployer itself.
+        Only works in mirrored deployment mode.
+
+        Args:
+            timestamps:
+                Show timestamps in the logs.
+            tail:
+                Number of lines to show from the end of the logs.
+            since:
+                Show logs since the given epoch in seconds.
+            follow:
+                Whether to follow the logs.
+
+        Returns:
+            The logs as a byte string or a generator yielding byte strings if follow is True.
+
+        Raises:
+            UnsupportedError:
+                If endoscopy is not supported in the current environment.
+            OperationError:
+                If the deployer fails to get logs.
+
+        """
+        self_container = self._find_self_container_for_endoscopy()
+
+        logs_options = {
+            "timestamps": timestamps,
+            "tail": tail if tail >= 0 else None,
+            "since": since,
+            "follow": follow,
+        }
+
+        try:
+            output = self_container.logs(
+                stream=follow,
+                **logs_options,
+            )
+        except podman.errors.APIError as e:
+            msg = f"Failed to fetch logs for self Container {self_container.short_id}{_detail_api_call_error(e)}"
+            raise OperationError(msg) from e
+        else:
+            if not follow:
+                result = bytearray()
+                for chunk in output:
+                    result.extend(chunk)
+                return result.decode("utf-8")
+            return output
+
+    def _endoscopic_exec(
+        self,
+        detach: bool = True,
+        command: list[str] | None = None,
+        args: list[str] | None = None,
+    ) -> WorkloadExecStream | bytes | str:
+        """
+        Execute a command in the deployer itself.
+        Only works in mirrored deployment mode.
+
+        Args:
+            detach:
+                Whether to detach from the command.
+            command:
+                The command to execute.
+                If not specified, use /bin/sh and implicitly attach.
+            args:
+                The arguments to pass to the command.
+
+        Returns:
+            If detach is False, return a WorkloadExecStream.
+            otherwise, return the output of the command as a byte string or string.
+
+        Raises:
+            UnsupportedError:
+                If endoscopy is not supported in the current environment.
+            OperationError:
+                If the deployer fails to execute the command.
+
+        """
+        self_container = self._find_self_container_for_endoscopy()
+
+        attach = not detach or not command
+        exec_options = {
+            "stdout": True,
+            "stderr": True,
+            "stdin": attach,
+            "socket": attach,
+            "tty": attach,
+            "cmd": [*command, *(args or [])] if command else ["/bin/sh"],
+        }
+
+        try:
+            _, output = self_container.exec_run(
+                detach=False,
+                **exec_options,
+            )
+        except podman.errors.APIError as e:
+            msg = f"Failed to exec command in self Container {self_container.short_id}{_detail_api_call_error(e)}"
+            raise OperationError(msg) from e
+        else:
+            if not attach:
+                return output
+            return PodmanWorkloadExecStream(output)
+
+    def _endoscopic_inspect(self) -> str:
+        """
+        Inspect the deployer itself.
+        Only works in mirrored deployment mode.
+
+        Returns:
+            The inspection result.
+
+        Raises:
+            UnsupportedError:
+                If endoscopy is not supported in the current environment.
+            OperationError:
+                If the deployer fails to execute the command.
+
+        """
+        self_container = self._find_self_container_for_endoscopy()
+
+        c_attrs = self_container.attrs
+        # Mask sensitive environment variables
+        if "Env" in c_attrs["Config"]:
+            for i, env in enumerate(c_attrs["Config"]["Env"] or []):
+                env_name, _ = env.split("=", maxsplit=1)
+                if sensitive_env_var(env_name):
+                    c_attrs["Config"]["Env"][i] = f"{env_name}=******"
+
+        return safe_json(c_attrs, indent=2)
+
 
 def _has_restart_policy(
     container: podman.domain.containers.Container,

gpustack_runtime/detector/amd.py

@@ -108,7 +108,7 @@ class AMDDetector(Detector):
                     dev_uuid = f"GPU-{(asic_serial[2:]).lower()}"
                 else:
                     dev_uuid = f"GPU-{pyrocmsmi.rsmi_dev_unique_id_get(dev_idx)[2:]}"
-                dev_hsa_agent = hsa_agents.get(dev_uuid)
+                dev_hsa_agent = hsa_agents.get(dev_uuid, pyhsa.Agent())
 
                 dev_gpu_driver_info = pyamdsmi.amdsmi_get_gpu_driver_info(dev)
                 dev_driver_ver = dev_gpu_driver_info.get("driver_version")

gpustack_runtime/detector/hygon.py

@@ -108,7 +108,7 @@ class HygonDetector(Detector):
                 dev_index = dev_idx
 
                 dev_uuid = f"GPU-{pyrocmsmi.rsmi_dev_unique_id_get(dev_idx)[2:]}"
-                dev_hsa_agent = hsa_agents.get(dev_uuid)
+                dev_hsa_agent = hsa_agents.get(dev_uuid, pyhsa.Agent())
 
                 dev_name = dev_hsa_agent.name
                 if not dev_name: