Skip to content

Support active Docker contexts #198

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
ale-cci wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Support active Docker contexts #198

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
54209fc
fix: Manually retrieve the docker-host from the current context
ale-cci Apr 3, 2026
f2627dd
fix: load TLS vars from env
ale-cci May 13, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
25 changes: 20 additions & 5 deletions tox_docker/plugin.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@
from tox.tox_env.api import ToxEnv
from tox.tox_env.errors import Fail
import docker as docker_module
import docker.utils

from tox_docker.config import (
ContainerConfig,
Expand All @@ -40,6 +41,22 @@ class HealthCheckFailed(Exception):
pass


def docker_client():
# docker.from_env() ignores the current context configuration.
# https://github.com/docker/docker-py/issues/3146

ctx = docker_module.ContextAPI.get_current_context()

params = docker_module.utils.kwargs_from_env(os.environ)
params.setdefault("base_url", ctx.Host)

docker = docker_module.DockerClient(
version="auto",
**params,
)
return docker


def get_gateway_ip(container: Container) -> str:
gateway = os.getenv("TOX_DOCKER_GATEWAY")
if gateway:
Expand Down Expand Up @@ -124,7 +141,7 @@ def docker_build_or_pull(container_config: ContainerConfig) -> None:
def docker_pull(container_config: ContainerConfig) -> None:
assert container_config.image

docker = docker_module.from_env(version="auto")
docker = docker_client()

try:
docker.images.get(str(container_config.image))
Expand Down Expand Up @@ -163,7 +180,7 @@ def docker_run(
container_config: ContainerConfig,
running_containers: RunningContainers,
) -> Container:
docker = docker_module.from_env(version="auto")
docker = docker_client()

healthcheck: Dict[str, Union[List[str], int]] = {}
if container_config.healthcheck_cmd:
Expand Down Expand Up @@ -215,8 +232,6 @@ def docker_run(
def docker_health_check(
container_config: ContainerConfig, container: Container
) -> None:
docker = docker_module.from_env(version="auto")

if "Health" in container.attrs["State"]:
log(f"health check {container_config.name!r}")
while True:
Expand All @@ -241,7 +256,7 @@ def docker_stop(container_config: ContainerConfig, container: Container) -> None


def docker_get(container_config: ContainerConfig) -> Optional[Container]:
docker = docker_module.from_env(version="auto")
docker = docker_client()
try:
return docker.containers.get(container_config.runas_name)
except NotFound:
Expand Down