From 9ce2fc718a73c466c0e88028e2645490bc0daf7b Mon Sep 17 00:00:00 2001 From: Ash Wilson Date: Tue, 24 Mar 2015 11:18:15 -0400 Subject: [PATCH] Only fetch details when necessary. --- cloud/docker/docker.py | 16 +++++++++++----- 1 file changed, 11 insertions(+), 5 deletions(-) diff --git a/cloud/docker/docker.py b/cloud/docker/docker.py index 4a8f4fef15c..85eb0525a69 100644 --- a/cloud/docker/docker.py +++ b/cloud/docker/docker.py @@ -1119,15 +1119,17 @@ class DockerManager(object): repo_tags = [normalize_image(self.module.params.get('image'))] for i in self.client.containers(all=True): - details = self.client.inspect_container(i['Id']) - details = _docker_id_quirk(details) - - running_image = normalize_image(details['Config']['Image']) - running_command = i['Command'].strip() + details = None if name: matches = name in i.get('Names', []) else: + details = self.client.inspect_container(i['Id']) + details = _docker_id_quirk(details) + + running_image = normalize_image(details['Config']['Image']) + running_command = i['Command'].strip() + image_matches = running_image in repo_tags # if a container has an entrypoint, `command` will actually equal @@ -1137,6 +1139,10 @@ class DockerManager(object): matches = image_matches and command_matches if matches: + if not details: + details = self.client.inspect_container(i['Id']) + details = _docker_id_quirk(details) + deployed.append(details) return deployed