Skip to content

Use monolithic PUT to upload large layers to Amazon ECR#822

Merged
amisevsk merged 3 commits intokitops-ml:mainfrom
amisevsk:fix-ecr-push
Apr 10, 2025
Merged

Use monolithic PUT to upload large layers to Amazon ECR#822
amisevsk merged 3 commits intokitops-ml:mainfrom
amisevsk:fix-ecr-push

Conversation

@amisevsk
Copy link
Copy Markdown
Contributor

@amisevsk amisevsk commented Apr 9, 2025

Description

Amazon ECR registries return HTTP 201 after the first chunk in a multi-part upload, which is unexpected and causes the upload to fail. To avoid this, we can default to using a monolithic PUT.

Linked issues

Closes #821

@amisevsk amisevsk requested a review from gorkem April 9, 2025 21:45
Comment thread pkg/lib/repo/remote/upload-format.go Outdated
Comment thread pkg/lib/repo/remote/upload-format_test.go
Amazon ECR registries return HTTP 201 after the first chunk in a
multi-part upload, which is unexpected and causes the upload to fail.
To avoid this, we can default to using a monolithic PUT.

Signed-off-by: Angel Misevski <amisevsk@gmail.com>
Signed-off-by: Angel Misevski <amisevsk@gmail.com>
Since URLs are case-insensitive

Signed-off-by: Angel Misevski <amisevsk@gmail.com>
@amisevsk amisevsk merged commit 84d6d3d into kitops-ml:main Apr 10, 2025
4 checks passed
@amisevsk amisevsk deleted the fix-ecr-push branch April 10, 2025 23:47
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.

Pushing ModelKits to ECR fails

2 participants