-
Notifications
You must be signed in to change notification settings - Fork 687
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Define image-related action nomenclature #467
Comments
On Fri, Nov 18, 2016 at 12:30:13PM -0800, Stephen Day wrote:
+1 to defining these sorts of terms. The spec already uses “applying changesets” 1 for the operation that I'm not aware of clear wording in the spec analagous to image-tool's “verify” seems pretty clear to me, and the spec's key words 4 and |
I've taken a stab at this in #554. |
so we want to go with "pack" and "unpack" vs "create" and "apply"? |
On Wed, Mar 08, 2017 at 11:18:56AM -0800, Vincent Batts wrote:
so we want to go with "pack" and "unpack" vs "create" and "apply"?
honestly, cleaning up the current wording is a separate task from
switching to new terms. These new terms are more confusing in the
sense of sounding like regular tar archives, which they aren't.
I don't think you need new words to distinguish from regular tar
archives. (un)packing can be generic words that apply to any media
type (which is what I'm aiming at in #554), and how they are
implemented should clearly (I hope) depend on the media type being
handled. Is there a reason to expect the application/x-tar algorithms
to apply out of the box to application/vnd.oci.image.layer.v1.tar
blobs? I don't think so. If that did work, we'd have just used
application/x-tar as our layer media type.
|
Should we close this issue and move discussion to PR #554 ? |
@RobDolinMS I'm not sure that #554 covers what I was looking for. I think this one is on my plate, but it will have to come after 1.0. |
The image-tools repository has a few different terms that are being applied to image layouts that seems unintuitive or confusing. To ensure that tools have operational parity, it seems like a solid idea to define the terminology for this common operations.
Specifically, we should define the high-level terms for "unpack", "create", "pack", "verify" (?).
This will make it clear what functionality the specification actually provides.
The text was updated successfully, but these errors were encountered: