NO-JIRA: chore(tests/containers): detect socket for ryuk's use with rootless podman on macOS

# Conflicts:
#	poetry.lock

Author: jiridanek

README.md

@@ -110,7 +110,7 @@ DOCKER_HOST=unix:///run/user/$UID/podman/podman.sock poetry run pytest tests/con
 # Mac OS
 brew install podman
 podman machine init
-podman machine set --rootful
+podman machine set --rootful=false
 sudo podman-mac-helper install
 podman machine start
 poetry run pytest tests/containers --image quay.io/opendatahub/workbench-images@sha256:e98d19df346e7abb1fa3053f6d41f0d1fa9bab39e49b4cb90b510ca33452c2e4

poetry.lock

@@ -17,6 +17,7 @@ pytest-subtests = "^0.14.1"
 pyfakefs = "^5.7.4"
 testcontainers = "^4.9.1"
 docker = "^7.1.0"
+podman = "^5.2.0"
 pydantic = "^2.10.6"
 requests = "^2.32.3"
 

pyproject.toml

@@ -3,15 +3,19 @@
 import io
 import logging
 import os.path
+import platform
 import sys
 import tarfile
 import time
 from typing import Iterable, TYPE_CHECKING
 
+import podman
 import docker.client
 
 import testcontainers.core.container
 
+import tests.containers.pydantic_schemas
+
 if TYPE_CHECKING:
     from docker.models.containers import Container
 
@@ -158,12 +162,24 @@ def communicate(self, line_prefix=b"") -> int:
             raise RuntimeError("Hm could that really happen?")
         return self.poll()
 
-def get_socket_path(client: docker.client.DockerClient) -> str:
+def get_socket_path(docker_client: docker.client.DockerClient) -> str:
     """Determine the local socket path.
     This works even when `podman machine` with its own host-mounts is involved
     NOTE: this will not work for remote docker, but we will cross the bridge when we come to it"""
-    socket_path = _the_one(adapter.socket_path for adapter in client.api.adapters.values())
-    return socket_path
+    socket_path = _the_one(adapter.socket_path for adapter in docker_client.api.adapters.values())
+    if platform.system().lower() == 'linux':
+        return socket_path
+    if platform.system().lower() == 'darwin':
+        # we want to return the socket path that's valid from inside Podman Machine
+        # with rootful podman, both the host (ls) and podman machine (podman machine ssh ls) have it at /var/run/docker.sock
+        # but with rootless podman, the location on host is still the same while podman machine has it in /var/run/user/${PID}/podman/podman.sock
+        podman_client = podman.PodmanClient(base_url="http+unix://" + socket_path)
+        info = tests.containers.pydantic_schemas.PodmanInfo.model_validate(podman_client.info())
+        assert info.host.remoteSocket.exists, "Failed to determine the podman remote socket"
+        assert info.host.remoteSocket.path.startswith("unix://"), "Unexpected remote socket path"
+        return info.host.remoteSocket.path[len("unix://"):]
+
+    raise RuntimeError(f"Unsupported platform: {platform.system()=}")
 
 def get_container_pid(container: Container) -> int | None:
     """Get the network namespace of a Docker container."""

tests/containers/docker_utils.py

@@ -4,16 +4,16 @@
 import subprocess
 from typing import Callable
 
-import tests.containers.schemas
+import tests.containers.pydantic_schemas
 
 logging.basicConfig(level=logging.DEBUG)
 
-def open_ssh_tunnel(machine_predicate: Callable[[tests.containers.schemas.PodmanMachine], bool],
+def open_ssh_tunnel(machine_predicate: Callable[[tests.containers.pydantic_schemas.PodmanMachine], bool],
                     local_port: int, remote_port: int, remote_interface: str = "localhost") -> subprocess.Popen:
     # Load and parse the Podman machine data
     machine_names = subprocess.check_output(["podman", "machine", "list", "--quiet"], text=True).splitlines()
     json_data = subprocess.check_output(["podman", "machine", "inspect", *machine_names], text=True)
-    inspect = tests.containers.schemas.PodmanMachineInspect(machines=json.loads(json_data))
+    inspect = tests.containers.pydantic_schemas.PodmanMachineInspect(machines=json.loads(json_data))
     machines = inspect.machines
 
     machine = next((m for m in machines if machine_predicate(m)), None)

tests/containers/podman_machine_utils.py

@@ -0,0 +1,8 @@
+from .podman_info import PodmanInfo
+from .podman_machine_inspect import PodmanMachineInspect, PodmanMachine
+
+__all__ = [
+    PodmanInfo,
+    PodmanMachineInspect,
+    PodmanMachine,
+]