[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