[ARVADOS] updated: 8a9fe2e16f1203f303afabc8c88b6e1ded9cec57

git at public.curoverse.com git at public.curoverse.com
Wed Oct 15 09:20:38 EDT 2014


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

       via  8a9fe2e16f1203f303afabc8c88b6e1ded9cec57 (commit)
      from  96ca1cadc8231a7d2e324602b70c73a76745e5eb (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 8a9fe2e16f1203f303afabc8c88b6e1ded9cec57
Author: Brett Smith <brett at curoverse.com>
Date:   Wed Oct 15 09:22:40 2014 -0400

    4196: arv-keepdocker uses Docker 1.0 pull syntax.
    
    `docker pull -t` was the old way of pulling an image with a specific
    tag.  Docker 1.0 introduced `docker pull repo:tag`, and Docker 1.2
    removes support for `-t` completely.  We're well-standardized on 1.1+
    at this point, and considering an upgrade to 1.2+, so now's the time
    to switch.
    
    Closes #4196.

diff --git a/sdk/python/arvados/commands/keepdocker.py b/sdk/python/arvados/commands/keepdocker.py
index c091374..6fa1745 100644
--- a/sdk/python/arvados/commands/keepdocker.py
+++ b/sdk/python/arvados/commands/keepdocker.py
@@ -116,7 +116,8 @@ def stat_cache_name(image_file):
     return getattr(image_file, 'name', image_file) + '.stat'
 
 def pull_image(image_name, image_tag):
-    check_docker(popen_docker(['pull', '-t', image_tag, image_name]), "pull")
+    check_docker(popen_docker(['pull', '{}:{}'.format(image_name, image_tag)]),
+                 "pull")
 
 def save_image(image_hash, image_file):
     # Save the specified Docker image to image_file, then try to save its

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list