-
Notifications
You must be signed in to change notification settings - Fork 184
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Replace AWS CLI by S3CMD to fix an issue with MultiPart on GCS #112
base: master
Are you sure you want to change the base?
Replace AWS CLI by S3CMD to fix an issue with MultiPart on GCS #112
Conversation
Not 100% sure how I feel about replacing the official AWS S3 utilities with a 3rd-party tool. I have used Doesn't mean we cannot do it; just want to consider for a bit. In any case, CI is failing. It looks like it isn't quite a drop-in replacement. We would need to fix that first? |
@deitch Can you have a look at the tests, i am really not sure where to look at. The only obvious error i can see is :
But i am not sure that the cause of the failed test |
From the CI:
s3cmd is doing something odd... |
There is a number of issues here:
Those two at least will get you a valid request. The rest actually isn't an invalid AWS key, but confusing error messages. If we do this, you need to add parsing logic to |
Fix #110