Re: [jclouds-labs-openstack] Add Rackspace CDN UK/US Providers (#177)

2015-01-28 Thread Zack Shoylev 
Thanks!

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/177#issuecomment-71950194

Re: [jclouds-labs-openstack] Add Rackspace CDN UK/US Providers (#177)

2015-01-27 Thread Jeremy Daggett 
@andrewgaul Thanks again for the review! Pushed to **master** 8b17517
/cc @zack-shoylev 

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/177#issuecomment-71787786

Re: [jclouds-labs-openstack] Add Rackspace CDN UK/US Providers (#177)

2015-01-27 Thread Jeremy Daggett 
> +
> +   /**
> +* @return a {@link Properties} object containing the default provider 
> properties.
> +* This returns the credential type, service type, and configured regions.
> +*/
> +   public static Properties defaultProperties() {
> +  Properties properties = new Properties();
> +  properties.setProperty(CREDENTIAL_TYPE, 
> CloudIdentityCredentialTypes.API_KEY_CREDENTIALS);
> +  properties.setProperty(SERVICE_TYPE, ServiceType.CDN);
> +
> +  properties.setProperty(PROPERTY_REGIONS, "ORD,DFW,IAD,SYD,HKG");
> +  properties.setProperty(PROPERTY_REGION + ".ORD." + ISO3166_CODES, 
> "US-IL");
> +  properties.setProperty(PROPERTY_REGION + ".DFW." + ISO3166_CODES, 
> "US-TX");
> +  properties.setProperty(PROPERTY_REGION + ".IAD." + ISO3166_CODES, 
> "US-VA");
> +  properties.setProperty(PROPERTY_REGION + ".SYD." + ISO3166_CODES, 
> "AU-NSW");
> +  properties.setProperty(PROPERTY_REGION + ".HKG." + ISO3166_CODES, 
> "HK");

@andrewgaul Yes! UK is the special case. All of the other regions are available 
via the US auth endpoint. HTH!

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/177/files#r23667059

Re: [jclouds-labs-openstack] Add Rackspace CDN UK/US Providers (#177)

2015-01-27 Thread Andrew Gaul 
> +
> +   /**
> +* @return a {@link Properties} object containing the default provider 
> properties.
> +* This returns the credential type, service type, and configured regions.
> +*/
> +   public static Properties defaultProperties() {
> +  Properties properties = new Properties();
> +  properties.setProperty(CREDENTIAL_TYPE, 
> CloudIdentityCredentialTypes.API_KEY_CREDENTIALS);
> +  properties.setProperty(SERVICE_TYPE, ServiceType.CDN);
> +
> +  properties.setProperty(PROPERTY_REGIONS, "ORD,DFW,IAD,SYD,HKG");
> +  properties.setProperty(PROPERTY_REGION + ".ORD." + ISO3166_CODES, 
> "US-IL");
> +  properties.setProperty(PROPERTY_REGION + ".DFW." + ISO3166_CODES, 
> "US-TX");
> +  properties.setProperty(PROPERTY_REGION + ".IAD." + ISO3166_CODES, 
> "US-VA");
> +  properties.setProperty(PROPERTY_REGION + ".SYD." + ISO3166_CODES, 
> "AU-NSW");
> +  properties.setProperty(PROPERTY_REGION + ".HKG." + ISO3166_CODES, 
> "HK");

Should SYD and HKG be here?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/177/files#r23666949

[jclouds-labs-openstack] Add Rackspace CDN UK/US Providers (#177)

2015-01-27 Thread Jeremy Daggett 
This PR adds support for the Early Access Rackspace CDN US/UK Providers, which 
is based on the OpenStack Poppy API.
You can view, comment on, or merge this pull request online at:

  https://github.com/jclouds/jclouds-labs-openstack/pull/177

-- Commit Summary --

  * Add Rackspace CDN UK/US Providers

-- File Changes --

M pom.xml (2)
A rackspace-cdn-uk/README.md (7)
A rackspace-cdn-uk/pom.xml (125)
A 
rackspace-cdn-uk/src/main/java/org/jclouds/rackspace/cdn/uk/CDNUKProviderMetadata.java
 (116)
A 
rackspace-cdn-uk/src/test/java/org/jclouds/rackspace/cdn/uk/CDNUKProviderTest.java
 (28)
A rackspace-cdn-uk/src/test/resources/logback.xml (56)
A rackspace-cdn-us/README.md (7)
A rackspace-cdn-us/pom.xml (125)
A 
rackspace-cdn-us/src/main/java/org/jclouds/rackspace/cdn/us/CDNUSProviderMetadata.java
 (121)
A 
rackspace-cdn-us/src/test/java/org/jclouds/rackspace/cdn/us/CDNUSProviderTest.java
 (28)
A rackspace-cdn-us/src/test/resources/logback.xml (56)

-- Patch Links --

https://github.com/jclouds/jclouds-labs-openstack/pull/177.patch
https://github.com/jclouds/jclouds-labs-openstack/pull/177.diff

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/177