incubator-deltacloud-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "marios@redhat.com" <mandr...@redhat.com>
Subject Re: [PATCH 1/4] Adds blob streaming uploads using PUT (stream through thin+deltacloud)
Date Thu, 09 Jun 2011 13:26:00 GMT
On 09/06/11 16:11, David Lutterkort wrote:
> On Thu, 2011-06-09 at 12:11 +0300, marios@redhat.com wrote:
>>>> +          #s3.amazonaws.com
>>>> +          provider = "https://#{Deltacloud::Drivers::driver_config[:ec2][:entrypoints]['s3']['us-east-1']}"
>>>
>>> How about making this configurable by headers o matrix params? Like I want to
use different provider
>>> than us-east-1 ?
>>>
>>
>> ===>  problem is that we really MUST use 'https://s3.amazonaws.com' (i.e.
>> the us-east endpoint) for putting blobs. Amazon redirects your put to
>> the appropriate endpoint depending on your bucket location (e.g. to
>> s3-eu-west-1.amazonaws.com for European buckets). I tried putting
>> directly to the European endpoint but failed. The 'endpoint_for_service'
>> method checks
>> "(Thread.current[:provider] || ENV['API_PROVIDER'] || DEFAULT_REGION) "
>> for the endpoint.
>
> If it turns out that we need to do it with the same URL for all regions,
> you should put a comment there; otherwise, somebody else might try and
> 'fix' it by using endpoint_for_service.
>
> David
>
>

I was wrong about this - last time I tried to specify something like: 
bucketname.s3-eu-west-1.amazonaws.com  (i.e. put to a european endpoint) 
it failed - it may have been some network issue or more likely a silly 
error - sorry about that. I will change the code to use 
'endpoint_for_service' before I push.

Mime
View raw message