New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Align endpoint status with rest of spec #555

Merged

sudo-bmitch merged 1 commit into opencontainers:main from sudo-bmitch:pr-endpoint-status

Sep 26, 2024

Contributor

sudo-bmitch commented Sep 25, 2024

Fixes #554. This aligns the endpoints table with the earlier definitions in the spec:

Added the 413 status to deleting manifests. (Missed in Document manifest size limit recommendation #293)
Added the 202 status to cross repository blob mounts.
Aligned the errors for deleting tags, manifests, and blobs, when deletes are not enabled, for consistency.
Reordered the paragraphs in "Deleting tags" because that was a weird ordering.


          Align endpoint status with rest of spec

1a6e7d3

Signed-off-by: Brandon Mitchell <git@bmitch.net>

sudo-bmitch requested review from dmcgowan, jzelinskie, jonjohnsonjr, jdolitsky, mikebrow and stevvooe as code owners

September 25, 2024 18:06

dmcgowan approved these changes

View reviewed changes

mikebrow approved these changes

View reviewed changes

Member

mikebrow left a comment

LGTM

sudo-bmitch merged commit 8376368 into opencontainers:main

4 checks passed

sudo-bmitch deleted the pr-endpoint-status branch

September 26, 2024 18:51

# for free to join this conversation on GitHub. Already have an account? # to comment

Reviewers

mikebrow mikebrow approved these changes

dmcgowan dmcgowan approved these changes

jzelinskie Awaiting requested review from jzelinskie jzelinskie is a code owner

jonjohnsonjr Awaiting requested review from jonjohnsonjr jonjohnsonjr is a code owner

jdolitsky Awaiting requested review from jdolitsky jdolitsky is a code owner

stevvooe Awaiting requested review from stevvooe stevvooe is a code owner

Labels

None yet