Skip to content
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

_catalog reserved reference missing from the current spec #306

Closed
SteveLasker opened this issue Nov 4, 2021 · 5 comments
Closed

_catalog reserved reference missing from the current spec #306

SteveLasker opened this issue Nov 4, 2021 · 5 comments

Comments

@SteveLasker
Copy link
Contributor

PR #69 moved the _catalog api to reserved, to assure current implementations didn't get clobbered with a new implementation of the same API.
This reference was lost in 1.0 version of the spec.

Opening an issue to get agreement on where we'd like to account for reserved elements of the spec.

@vbatts
Copy link
Member

vbatts commented Nov 22, 2021

uuuhmm. I'm missing something. Your PR removed it, but now this is a discussion to re-add it as reserved?

@SteveLasker
Copy link
Contributor Author

PR #69 had text around it being reserved

The _catalog api is reserved for historical usage. Registries MAY implement _catalog, but are NOT required.

I was looking for this reference to describe what some registries have implemented as an option for repo listing. I'd like to introduce a new pattern but wanted to refer to _catalog as not being supported.
I'm hopeful we can use #111 as a way to get a scoped list of repos.

If folks agree with restoring the reserved _catalog reference, I can make the pr to bring it back as reserved.

@vbatts
Copy link
Member

vbatts commented Dec 9, 2021

the existence and implementation of _catalog was a wart, and I don't think reserving it fixes or changes anything.
The proper fix would be to provide proper way for search/listing to be possible.

@SteveLasker
Copy link
Contributor Author

I completely agree we should implement a new listing and search API, that acknowledges the org and repo permission boundaries registries have implemented. See #71 for the most recent discussion, which also references #111 as the base.
But, the _catalog wart was part of the spec, and several registries implemented it. The line that got lost was simply capturing _catalog is reserved so it's not clobbered with new implementations.

@sajayantony
Copy link
Member

sajayantony commented Aug 11, 2022

I think we can close this now as it is marked reserved here - https://github.com/opencontainers/distribution-spec/blob/b1257f661d893af1dd0b51ba2d75cc830a1a1c46/extensions/README.md#reserved-namespaces

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants