Bug #8315

[SDKs] arv-copy fails to copy a Docker image specified with repo:tag syntax

Added by Sarah Guthrie over 3 years ago. Updated over 3 years ago.

Status:
New
Priority:
Normal
Assigned To:
-
Category:
SDKs
Target version:
Start date:
01/27/2016
Due date:
% Done:

0%

Estimated time:
Story points:
0.5

Description

Since docker uses colons to specify tags, when writing pipeline templates, a developer will use "docker_image":"repo/image:tag" in runtime_constraints. This format breaks when using arv copy, since docker images and tags are space separated in arvados, causing the relevant docker images to not be copied.


Related issues

Related to Arvados - Feature #8333: [SDKs] `arv keep docker` supports `repo:tag` image name schemeResolved02/01/2016

History

#1 Updated by Brett Smith over 3 years ago

  • Subject changed from Support docker versioning using colons to [SDKs] arv-copy fails to copy a Docker image specified with repo:tag syntax
  • Description updated (diff)
  • Category set to SDKs
  • Story points set to 0.5

I'll make a separate issue for the arv keep docker request.

I'm going out on a limb and saying this is probably .5 points because it should just be a parsing error.

#2 Updated by Brett Smith over 3 years ago

  • Target version set to Arvados Future Sprints

#3 Updated by Brett Smith over 3 years ago

Sally,

It would be helpful to have the UUID of the pipeline template you tried to copy. That would help to see whether the problem was caused by an earlier bug; e.g., from #7154.

#4 Updated by Brett Smith over 3 years ago

Nevermind, confirmed the bug is in fact in arv-copy. Probably because this syntax wasn't expected to work: we have a separate docker_image_tag constraint to specify the tag. But we should probably move to more the way you're doing it.

(Developers: See copy_docker_images in arv_copy.py. We can extend this backward-compatibly by trying to parse the tag out from docker_image when docker_image_tag isn't set. latest should remain the ultimate default.)

Also available in: Atom PDF