Skip to content

use aws cli for uploads >1GB#132

Merged
mxcl merged 12 commits intomainfrom
fix-large-uploads
Jun 14, 2023
Merged

use aws cli for uploads >1GB#132
mxcl merged 12 commits intomainfrom
fix-large-uploads

Conversation

@jhheider
Copy link
Contributor

@jhheider jhheider commented Jun 14, 2023

closes #130
closes pkgxdev/pantry#2167
closes pkgxdev/pantry#2177

proof: https://github.com/teaxyz/pantry/actions/runs/5261282714/jobs/9512823288

(linux+x86-64 uploads are 2.5GB (gz) and 1.5GB (xz))

@jhheider jhheider requested a review from mxcl June 14, 2023 04:32
@mxcl
Copy link
Contributor

mxcl commented Jun 14, 2023

Why only use it conditionally? Maintaining 2 paths vs 1 path is not ideal.

@jhheider
Copy link
Contributor Author

It's true. I considered just punting for all files. Probably simpler.

@mxcl mxcl merged commit 1294eb9 into main Jun 14, 2023
@mxcl mxcl deleted the fix-large-uploads branch June 14, 2023 18:41
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.

❌ build issues: github.com/coqui-ai/TTS Some bottles are too big to upload as a UInt8Array ❌ build issues: github.com/coqui-ai/TTS

2 participants