Skip to content

Use monolithic uploads when pushing to Google Artifact Registry#780

Merged
amisevsk merged 2 commits intokitops-ml:mainfrom
amisevsk:google-artifact-registry-support
Feb 25, 2025
Merged

Use monolithic uploads when pushing to Google Artifact Registry#780
amisevsk merged 2 commits intokitops-ml:mainfrom
amisevsk:google-artifact-registry-support

Conversation

@amisevsk
Copy link
Copy Markdown
Contributor

Description

Google Artifact Registry does not support chunked uploads and instead requires monolithic uploads. Handle pushes to the registry similar to how we handle ghcr.io (matching .pkg.dev)

Note: I haven't been able to test this directly, yet, as I don't have access to a Google Artifact Registry at the moment. I'll test to double check before merging

Linked issues

Closes #777

Google Artifact Registry does not support chunked uploads and instead
requires monolithic uploads. Handle pushes to the registry similar to
how we handle ghcr.io (matching .pkg.dev)
@amisevsk amisevsk requested a review from gorkem February 25, 2025 02:11
Copy link
Copy Markdown
Member

@gorkem gorkem left a comment

Choose a reason for hiding this comment

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

LGTM

@gorkem
Copy link
Copy Markdown
Member

gorkem commented Feb 25, 2025

Do we have tests covering upload-format? The list of special treatments may be growing.

@amisevsk
Copy link
Copy Markdown
Contributor Author

Do we have tests covering upload-format? The list of special treatments may be growing

Good idea, I'll add some

@amisevsk amisevsk force-pushed the google-artifact-registry-support branch from 75dcb76 to 5d1af89 Compare February 25, 2025 17:35
@amisevsk
Copy link
Copy Markdown
Contributor Author

Tested on Google Artifact Registry and was able to push a 1GB layer without issue.

I logged in using an access_token, as Kit currently does not support auth helpers.

@amisevsk amisevsk merged commit e3ec1f2 into kitops-ml:main Feb 25, 2025
@amisevsk amisevsk deleted the google-artifact-registry-support branch February 25, 2025 21:34
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.

Kit cannot upload large files to Google Artifact Registry

2 participants