Skip to content

Add OCI-Chunk-Min-Length header#391

Merged
sudo-bmitch merged 1 commit intoopencontainers:mainfrom
sudo-bmitch:pr-chunk-min-size
Mar 17, 2023
Merged

Add OCI-Chunk-Min-Length header#391
sudo-bmitch merged 1 commit intoopencontainers:mainfrom
sudo-bmitch:pr-chunk-min-size

Conversation

@sudo-bmitch
Copy link
Copy Markdown
Contributor

This should fix #374. It adds a new OCI-Chunk-Min-Bytes header on the blob POST method response. This allows the client to push chunks of at least that size, which is needed for some S3 backends.

@sudo-bmitch sudo-bmitch force-pushed the pr-chunk-min-size branch 2 times, most recently from 89e6342 to 5a465f1 Compare March 15, 2023 14:57
@sudo-bmitch sudo-bmitch marked this pull request as ready for review March 15, 2023 17:58
Copy link
Copy Markdown
Member

@mikebrow mikebrow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

comment and a question on remainder less than min..

jdolitsky
jdolitsky previously approved these changes Mar 16, 2023
Signed-off-by: Brandon Mitchell <git@bmitch.net>
Copy link
Copy Markdown
Member

@mikebrow mikebrow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sudo-bmitch sudo-bmitch changed the title Add OCI-Chunk-Min-Bytes header Add OCI-Chunk-Min-Length header Mar 16, 2023
@sudo-bmitch
Copy link
Copy Markdown
Contributor Author

@jay-dee7 any concerns with the latest changes? If not, I'll merge this.

@jay-dee7
Copy link
Copy Markdown

@sudo-bmitch I'm super happy and excited about this change. Please merge it :) and thanks a ton for getting this done the entire OCI team ❤️ ❤️

@sudo-bmitch sudo-bmitch merged commit 2c8ec1a into opencontainers:main Mar 17, 2023
@sudo-bmitch sudo-bmitch deleted the pr-chunk-min-size branch March 17, 2023 11:35
@jdolitsky jdolitsky mentioned this pull request Apr 19, 2023
@sudo-bmitch sudo-bmitch mentioned this pull request Feb 1, 2024
8 tasks
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

Successfully merging this pull request may close these issues.

Option to set custom layer size via environment variables

4 participants