Project

General

Profile

Actions

Bug #8315

open

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

Added by Sarah Guthrie about 8 years ago. Updated about 2 months ago.

Status:
New
Priority:
Normal
Assigned To:
-
Category:
SDKs
Target version:
Story points:
0.5
Release:
Release relationship:
Auto

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 schemeResolvedTom Clegg02/01/2016Actions
Actions #1

Updated by Brett Smith about 8 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.

Actions #2

Updated by Brett Smith about 8 years ago

  • Target version set to Arvados Future Sprints
Actions #3

Updated by Brett Smith about 8 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.

Actions #4

Updated by Brett Smith about 8 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.)

Actions #5

Updated by Ward Vandewege almost 3 years ago

  • Target version deleted (Arvados Future Sprints)
Actions #6

Updated by Peter Amstutz about 1 year ago

  • Release set to 60
Actions #7

Updated by Peter Amstutz about 2 months ago

  • Target version set to Future
Actions

Also available in: Atom PDF