Re: [jclouds/jclouds] JCLOUDS-1217 Add Canada and London regions to AWS (#1082)

2017-03-29 Thread Ignasi Barrera 
Closed #1082.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1082#event-1019906728

Re: [jclouds/jclouds] JCLOUDS-1217 Add Canada and London regions to AWS (#1082)

2017-03-29 Thread Ignasi Barrera 
Thanks @q225zhan! I've pushed the changes to 
[master](http://git-wip-us.apache.org/repos/asf/jclouds/commit/96a1c442) and 
[2.0.x](http://git-wip-us.apache.org/repos/asf/jclouds/commit/73875884) and 
amended the commit message to include the JIRA issues.
I've also added the Ohio region in an [additional 
commit](http://git-wip-us.apache.org/repos/asf/jclouds/commit/04ab255d) where I 
also added the new ISO-3166 codes to the test classes that reference them.
Thanks!

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1082#issuecomment-290006769

[jclouds/jclouds] JCLOUDS-1217 Add Canada and London regions to AWS (#1082)

2017-03-28 Thread Michelle Zhang 
I find the answer here:
http://docs.aws.amazon.com/AWSJavaSDK/latest/javadoc/com/amazonaws/services/s3/model/Region.html#EU_London
and
https://sourcegraph.com/github.com/aws/aws-sdk-java/-/blob/aws-java-sdk-s3/src/main/java/com/amazonaws/services/s3/model/Region.java#L166:12
You can view, comment on, or merge this pull request online at:

  https://github.com/jclouds/jclouds/pull/1082

-- Commit Summary --

  * Add Canada and London regions to AWS

-- File Changes --

M apis/sts/src/main/java/org/jclouds/aws/domain/Region.java (36)
M 
providers/aws-ec2/src/main/java/org/jclouds/aws/ec2/AWSEC2ProviderMetadata.java 
(2)
M 
providers/aws-s3/src/main/java/org/jclouds/aws/s3/AWSS3ProviderMetadata.java (9)

-- Patch Links --

https://github.com/jclouds/jclouds/pull/1082.patch
https://github.com/jclouds/jclouds/pull/1082.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1082