Bug #10969
closed
arv keep docker to check for valid Docker image and version
Added by Tom Morris almost 8 years ago.
Updated almost 8 years ago.
Estimated time:
(Total: 0.00 h)
Description
Add list of supported Docker image format versions to API server discovery document.
arv keep docker to check API server versions and fail if the on-disk format of uploaded image is incompatible.
- Description updated (diff)
- Story points set to 1.0
- Category set to Docker
- Assigned To set to Tom Clegg
The two relevant formats are "image manifest V 2, schema 1" and "image manifest V 2, schema 2". We could label those "v2s1" and "v2s2".
Proposed addition to source:services/api/config/application.default.yml:
# List of supported docker image formats that compute nodes are able
# to run. `arv keep docker` will error out if a user tries to store
# an image with an unsupported format. With the default value (an
# empty array), `arv keep docker` will skip the compatibility check
# and display a warning message to that effect.
#
# Example for sites running docker < 1.10:
# ["v2s1"]
#
# Example for sites running docker >= 1.10:
# ["v2s2"]
docker_image_formats: []
The addition looks fine, and the specification of format as well.
I think the default should be v2s2, not empty. And we should add a note to the migration wiki page for people who are upgrading.
Hm. Looks like "registry v1" to "registry v2" is the change we're dealing with here, not manifest format. New version:
# List of supported Docker Registry image formats that compute nodes
# are able to use. `arv keep docker` will error out if a user tries
# to store an image with an unsupported format. Use an empty array
# to skip the compatibility check (and display a warning message to
# that effect).
#
# Example for sites running docker < 1.10: ["v1"]
# Example for sites running docker >= 1.10: ["v2"]
# Example for disabling check: []
docker_image_formats: ["v2"]
- Status changed from New to In Progress
This lgtm, please merge. Thanks!
- Status changed from In Progress to Resolved
Applied in changeset arvados|commit:74a9decf66adc216d3d8bbb7ee8f6e3704d9590d.
Also available in: Atom
PDF