-
Notifications
You must be signed in to change notification settings - Fork 205
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
Spec seems to be missing cross-repo mounting entirely #186
Comments
I think the omission is intentional, as mounting isn't fully worked out for 1.0 release. Was discussed that this, along with extensions, and other topics can be reworked/re-added for subsequent releases. It is intended that detail.md will be referenced in spec.md prior to 1.0 |
Interesting! Do you remember where the discussion took place? Have a link handy? |
@jonjohnsonjr It was a vidchat several weeks ago between maintainers related to getting this PR merged: #178 Since we didn't want to lose useful bits, big chunk was saved off into details.md, although it shouldn't be considered "official". Working on adding that language in #188 @vbatts - correct me if I'm wrong about punting extensions stuff until after 1.0 |
I don't think that is accurate. Mounting is fully defined and regularly used today, I haven't heard of any aspect of it that was considered to be in-progress. |
@dmcgowan ok, lets add it back. Which section should it be defined under? |
Next to regular blob push. Used when initiating a new upload |
@jonjohnsonjr @dmcgowan please see #204 |
Resolved by #206 |
This section from the docker spec seems to have been dropped: https://github.com/docker/distribution/blob/5cb406d511b7b9163bff9b6439072e4892e5ae3b/docs/spec/api.md#cross-repository-blob-mount
I see that some info is still present in detail.md, which isn't referenced by spec.md at all: https://github.com/opencontainers/distribution-spec/blob/master/detail.md
Was this omission intentional?
The text was updated successfully, but these errors were encountered: