[ARVADOS] updated: 03184efc5c88fee5b2d956daa37a2f686b2eed60

git at public.curoverse.com git at public.curoverse.com
Tue Jun 10 17:42:30 EDT 2014


Summary of changes:
 sdk/python/arvados/commands/keepdocker.py | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

       via  03184efc5c88fee5b2d956daa37a2f686b2eed60 (commit)
      from  bfcf53c5b735b3872075d6ef1295c7115c112d53 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 03184efc5c88fee5b2d956daa37a2f686b2eed60
Author: Brett Smith <brett at curoverse.com>
Date:   Tue Jun 10 17:43:25 2014 -0400

    2879: Bugfix how arv-keepdocker ignores the `docker images` header.

diff --git a/sdk/python/arvados/commands/keepdocker.py b/sdk/python/arvados/commands/keepdocker.py
index f4ccb2a..39dc664 100644
--- a/sdk/python/arvados/commands/keepdocker.py
+++ b/sdk/python/arvados/commands/keepdocker.py
@@ -71,8 +71,9 @@ def check_docker(proc, description):
 def _get_docker_images():
     # Yield a DockerImage tuple for each installed image.
     list_proc = popen_docker(['images', '--no-trunc'], stdout=subprocess.PIPE)
-    next(list_proc.stdout)  # Ignore the header line
-    for line in list_proc.stdout:
+    list_output = iter(list_proc.stdout)
+    next(list_output)  # Ignore the header line
+    for line in list_output:
         words = line.split()
         size_index = len(words) - 2
         repo, tag, imageid = words[:3]

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list