From 40a62f8ba2cf650b497aa3cf8cdb47598dd0dd3d Mon Sep 17 00:00:00 2001 From: Nick Yang Date: Thu, 3 Oct 2024 15:16:16 -0500 Subject: [PATCH] docker: allow `import docker` in `reset` to fail (#268) * if the user hasn't installed the `[docker]` extra, they won't have `docker` to `import`, so don't make `import docker` required in the docker driver's `reset`. * fixes #166 --- src/molecule_plugins/docker/driver.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/molecule_plugins/docker/driver.py b/src/molecule_plugins/docker/driver.py index a22a6e5..894eee6 100644 --- a/src/molecule_plugins/docker/driver.py +++ b/src/molecule_plugins/docker/driver.py @@ -255,7 +255,11 @@ def sanity_checks(self): self._passed_sanity = True def reset(self): - import docker + # maybe use self.sanity_check instead? + try: + import docker + except ImportError: + return client = docker.from_env() for c in client.containers.list(filters={"label": "owner=molecule"}):