From 1c4023031064ef60de6316b5a46c3925cc92192e Mon Sep 17 00:00:00 2001 From: guppy0130 Date: Tue, 17 Sep 2024 20:01:41 -0500 Subject: [PATCH] docker: allow `import docker` in `reset` to fail * 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"}):