From b5419a026796c56df32dbd72569690c60321885b Mon Sep 17 00:00:00 2001 From: Adam Williamson Date: Tue, 13 Oct 2015 22:33:46 -0700 Subject: fix #2043: strip empty dict from end of 'pull' stream When pulling an image using Docker 1.8, it seems the output JSON stream has an empty dict at the very end. This causes ansible to fail when pulling an image, as it's expecting a status message in that dict which it uses to determine whether it had to download the image or not. As a bit of an ugly hack for that which remains backward compatible, try the last item in the stream, and if it's an empty dict, take the last-but-one item instead. The strip() is needed as the exact value appears to be '{}/r/n'; we could just match that, but it seems like the kind of thing where maybe it'd happen to just be '{}/n' or '{}' or something in some cases, so let's just use strip() in case. --- cloud/docker/docker.py | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/cloud/docker/docker.py b/cloud/docker/docker.py index ab9f13f5..f299454f 100644 --- a/cloud/docker/docker.py +++ b/cloud/docker/docker.py @@ -1237,6 +1237,11 @@ class DockerManager(object): changes = list(self.client.pull(image, tag=tag, stream=True, **extra_params)) try: last = changes[-1] + # seems Docker 1.8 puts an empty dict at the end of the + # stream; catch that and get the previous instead + # https://github.com/ansible/ansible-modules-core/issues/2043 + if last.strip() == '{}': + last = changes[-2] except IndexError: last = '{}' status = json.loads(last).get('status', '') -- cgit v1.2.1