You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@jclouds.apache.org by Jeremy Daggett <no...@github.com> on 2014/10/09 23:20:45 UTC
[jclouds] Import rackspace-cloudfiles from labs (#565)
This PR imports the `rackspace-cloudfiles` API and `rackspace-cloudfiles-uk/s` providers from the `jclouds-labs-openstack` repository.
You can merge this Pull Request by running:
git pull https://github.com/rackspace/jclouds add-rackspace-cloudfiles
Or you can view, comment on it, or merge it online at:
https://github.com/jclouds/jclouds/pull/565
-- Commit Summary --
* Import rackspace-cloudfiles from labs
-- File Changes --
M apis/pom.xml (1)
A apis/rackspace-cloudfiles/README.md (22)
A apis/rackspace-cloudfiles/pom.xml (170)
A apis/rackspace-cloudfiles/src/main/java/org/jclouds/rackspace/cloudfiles/v1/CloudFilesApi.java (51)
A apis/rackspace-cloudfiles/src/main/java/org/jclouds/rackspace/cloudfiles/v1/CloudFilesApiMetadata.java (101)
A apis/rackspace-cloudfiles/src/main/java/org/jclouds/rackspace/cloudfiles/v1/binders/BindCDNPurgeEmailAddressesToHeaders.java (53)
A apis/rackspace-cloudfiles/src/main/java/org/jclouds/rackspace/cloudfiles/v1/config/CloudFilesHttpApiModule.java (50)
A apis/rackspace-cloudfiles/src/main/java/org/jclouds/rackspace/cloudfiles/v1/domain/CDNContainer.java (262)
A apis/rackspace-cloudfiles/src/main/java/org/jclouds/rackspace/cloudfiles/v1/features/CDNApi.java (204)
A apis/rackspace-cloudfiles/src/main/java/org/jclouds/rackspace/cloudfiles/v1/functions/ParseCDNContainerFromHeaders.java (81)
A apis/rackspace-cloudfiles/src/main/java/org/jclouds/rackspace/cloudfiles/v1/functions/ParseCDNContainerURIFromHeaders.java (41)
A apis/rackspace-cloudfiles/src/main/java/org/jclouds/rackspace/cloudfiles/v1/functions/RegionToCDNEndpoint.java (73)
A apis/rackspace-cloudfiles/src/main/java/org/jclouds/rackspace/cloudfiles/v1/handlers/CloudFilesErrorHandler.java (91)
A apis/rackspace-cloudfiles/src/main/java/org/jclouds/rackspace/cloudfiles/v1/options/ListCDNContainerOptions.java (86)
A apis/rackspace-cloudfiles/src/main/java/org/jclouds/rackspace/cloudfiles/v1/options/UpdateCDNContainerOptions.java (177)
A apis/rackspace-cloudfiles/src/main/java/org/jclouds/rackspace/cloudfiles/v1/reference/CloudFilesConstants.java (31)
A apis/rackspace-cloudfiles/src/main/java/org/jclouds/rackspace/cloudfiles/v1/reference/CloudFilesHeaders.java (44)
A apis/rackspace-cloudfiles/src/main/resources/META-INF/services/org.jclouds.apis.ApiMetadata (18)
A apis/rackspace-cloudfiles/src/test/java/org/jclouds/rackspace/cloudfiles/v1/CloudFilesApiLiveTest.java (27)
A apis/rackspace-cloudfiles/src/test/java/org/jclouds/rackspace/cloudfiles/v1/CloudFilesApiMetadataTest.java (34)
A apis/rackspace-cloudfiles/src/test/java/org/jclouds/rackspace/cloudfiles/v1/binders/BindCDNPurgeEmailAddressesToHeadersMockTest.java (74)
A apis/rackspace-cloudfiles/src/test/java/org/jclouds/rackspace/cloudfiles/v1/blobstore/CloudFilesRegionScopedBlobStoreContextLiveTest.java (42)
A apis/rackspace-cloudfiles/src/test/java/org/jclouds/rackspace/cloudfiles/v1/blobstore/integration/CloudFilesBlobIntegrationLiveTest.java (39)
A apis/rackspace-cloudfiles/src/test/java/org/jclouds/rackspace/cloudfiles/v1/blobstore/integration/CloudFilesBlobLiveTest.java (39)
A apis/rackspace-cloudfiles/src/test/java/org/jclouds/rackspace/cloudfiles/v1/blobstore/integration/CloudFilesBlobSignerLiveTest.java (39)
A apis/rackspace-cloudfiles/src/test/java/org/jclouds/rackspace/cloudfiles/v1/blobstore/integration/CloudFilesContainerIntegrationLiveTest.java (39)
A apis/rackspace-cloudfiles/src/test/java/org/jclouds/rackspace/cloudfiles/v1/blobstore/integration/CloudFilesContainerLiveTest.java (56)
A apis/rackspace-cloudfiles/src/test/java/org/jclouds/rackspace/cloudfiles/v1/blobstore/integration/CloudFilesServiceIntegrationLiveTest.java (39)
A apis/rackspace-cloudfiles/src/test/java/org/jclouds/rackspace/cloudfiles/v1/features/CloudFilesAccountApiLiveTest.java (44)
A apis/rackspace-cloudfiles/src/test/java/org/jclouds/rackspace/cloudfiles/v1/features/CloudFilesBulkApiLiveTest.java (31)
A apis/rackspace-cloudfiles/src/test/java/org/jclouds/rackspace/cloudfiles/v1/features/CloudFilesCDNApiLiveTest.java (190)
A apis/rackspace-cloudfiles/src/test/java/org/jclouds/rackspace/cloudfiles/v1/features/CloudFilesCDNApiMockTest.java (523)
A apis/rackspace-cloudfiles/src/test/java/org/jclouds/rackspace/cloudfiles/v1/features/CloudFilesContainerApiLiveTest.java (31)
A apis/rackspace-cloudfiles/src/test/java/org/jclouds/rackspace/cloudfiles/v1/features/CloudFilesObjectApiLiveTest.java (31)
A apis/rackspace-cloudfiles/src/test/java/org/jclouds/rackspace/cloudfiles/v1/internal/BaseCloudFilesApiLiveTest.java (31)
A apis/rackspace-cloudfiles/src/test/java/org/jclouds/rackspace/cloudfiles/v1/options/UpdateCDNContainerOptionsTest.java (109)
A apis/rackspace-cloudfiles/src/test/resources/cdn_container_list.json (42)
A apis/rackspace-cloudfiles/src/test/resources/cdn_container_list_at.json (22)
A apis/rackspace-cloudfiles/src/test/resources/logback.xml (71)
M providers/pom.xml (2)
A providers/rackspace-cloudfiles-uk/README.md (21)
A providers/rackspace-cloudfiles-uk/pom.xml (175)
A providers/rackspace-cloudfiles-uk/src/main/java/org/jclouds/rackspace/cloudfiles/uk/CloudFilesUKProviderMetadata.java (121)
A providers/rackspace-cloudfiles-uk/src/main/resources/META-INF/services/org.jclouds.providers.ProviderMetadata (18)
A providers/rackspace-cloudfiles-uk/src/test/java/org/jclouds/rackspace/cloudfiles/uk/CloudFilesUKProviderTest.java (28)
A providers/rackspace-cloudfiles-uk/src/test/java/org/jclouds/rackspace/cloudfiles/uk/blobstore/integration/CloudFilesUKBlobIntegrationLiveTest.java (27)
A providers/rackspace-cloudfiles-uk/src/test/java/org/jclouds/rackspace/cloudfiles/uk/blobstore/integration/CloudFilesUKBlobLiveTest.java (27)
A providers/rackspace-cloudfiles-uk/src/test/java/org/jclouds/rackspace/cloudfiles/uk/blobstore/integration/CloudFilesUKBlobSignerLiveTest.java (27)
A providers/rackspace-cloudfiles-uk/src/test/java/org/jclouds/rackspace/cloudfiles/uk/blobstore/integration/CloudFilesUKContainerIntegrationLiveTest.java (27)
A providers/rackspace-cloudfiles-uk/src/test/java/org/jclouds/rackspace/cloudfiles/uk/blobstore/integration/CloudFilesUKContainerLiveTest.java (27)
A providers/rackspace-cloudfiles-uk/src/test/java/org/jclouds/rackspace/cloudfiles/uk/blobstore/integration/CloudFilesUKServiceIntegrationLiveTest.java (36)
A providers/rackspace-cloudfiles-uk/src/test/resources/logback.xml (71)
A providers/rackspace-cloudfiles-us/README.md (22)
A providers/rackspace-cloudfiles-us/pom.xml (175)
A providers/rackspace-cloudfiles-us/src/main/java/org/jclouds/rackspace/cloudfiles/us/CloudFilesUSProviderMetadata.java (143)
A providers/rackspace-cloudfiles-us/src/main/resources/META-INF/services/org.jclouds.providers.ProviderMetadata (18)
A providers/rackspace-cloudfiles-us/src/test/java/org/jclouds/rackspace/cloudfiles/us/CloudFilesUSProviderTest.java (28)
A providers/rackspace-cloudfiles-us/src/test/java/org/jclouds/rackspace/cloudfiles/us/blobstore/integration/CloudFilesUSBlobIntegrationLiveTest.java (27)
A providers/rackspace-cloudfiles-us/src/test/java/org/jclouds/rackspace/cloudfiles/us/blobstore/integration/CloudFilesUSBlobLiveTest.java (27)
A providers/rackspace-cloudfiles-us/src/test/java/org/jclouds/rackspace/cloudfiles/us/blobstore/integration/CloudFilesUSBlobSignerLiveTest.java (27)
A providers/rackspace-cloudfiles-us/src/test/java/org/jclouds/rackspace/cloudfiles/us/blobstore/integration/CloudFilesUSContainerIntegrationLiveTest.java (27)
A providers/rackspace-cloudfiles-us/src/test/java/org/jclouds/rackspace/cloudfiles/us/blobstore/integration/CloudFilesUSContainerLiveTest.java (27)
A providers/rackspace-cloudfiles-us/src/test/java/org/jclouds/rackspace/cloudfiles/us/blobstore/integration/CloudFilesUSServiceIntegrationLiveTest.java (36)
A providers/rackspace-cloudfiles-us/src/test/resources/logback.xml (71)
-- Patch Links --
https://github.com/jclouds/jclouds/pull/565.patch
https://github.com/jclouds/jclouds/pull/565.diff
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-pull-requests-java-6 #181](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests-java-6/181/) SUCCESS
This pull request looks good
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565#issuecomment-58579898
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by Adrian Cole <no...@github.com>.
> + <groupId>org.apache.jclouds.provider</groupId>
> + <artifactId>rackspace-cloudfiles-us</artifactId>
> + <version>2.0.0-SNAPSHOT</version>
> + <name>jclouds Rackspace Cloud Files US provider</name>
> + <description>OpenStack Object Storage implementation targeted to Rackspace Cloud Files US</description>
> + <packaging>bundle</packaging>
> +
> + <properties>
> + <!-- identity endpoint -->
> + <test.rackspace-cloudfiles-us.endpoint>https://identity.api.rackspacecloud.com/v2.0/</test.rackspace-cloudfiles-us.endpoint>
> + <test.rackspace-cloudfiles-us.api-version>1</test.rackspace-cloudfiles-us.api-version>
> + <test.rackspace-cloudfiles-us.build-version />
> + <test.rackspace-cloudfiles-us.identity>${test.rackspace-us.identity}</test.rackspace-cloudfiles-us.identity>
> + <test.rackspace-cloudfiles-us.credential>${test.rackspace-us.credential}</test.rackspace-cloudfiles-us.credential>
> + <jclouds.osgi.export>org.jclouds.rackspace.cloudfiles.us*;version="${project.version}"</jclouds.osgi.export>
> + <jclouds.osgi.import>
I think these are simplified in other poms
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565/files#r18675302
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-pull-requests #1273](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/1273/) SUCCESS
This pull request looks good
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565#issuecomment-58589856
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by Jeremy Daggett <no...@github.com>.
> + <groupId>org.apache.jclouds.provider</groupId>
> + <artifactId>rackspace-cloudfiles-us</artifactId>
> + <version>2.0.0-SNAPSHOT</version>
> + <name>jclouds Rackspace Cloud Files US provider</name>
> + <description>OpenStack Object Storage implementation targeted to Rackspace Cloud Files US</description>
> + <packaging>bundle</packaging>
> +
> + <properties>
> + <!-- identity endpoint -->
> + <test.rackspace-cloudfiles-us.endpoint>https://identity.api.rackspacecloud.com/v2.0/</test.rackspace-cloudfiles-us.endpoint>
> + <test.rackspace-cloudfiles-us.api-version>1</test.rackspace-cloudfiles-us.api-version>
> + <test.rackspace-cloudfiles-us.build-version />
> + <test.rackspace-cloudfiles-us.identity>${test.rackspace-us.identity}</test.rackspace-cloudfiles-us.identity>
> + <test.rackspace-cloudfiles-us.credential>${test.rackspace-us.credential}</test.rackspace-cloudfiles-us.credential>
> + <jclouds.osgi.export>org.jclouds.rackspace.cloudfiles.us*;version="${project.version}"</jclouds.osgi.export>
> + <jclouds.osgi.import>
Example?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565/files#r18675924
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by Adrian Cole <no...@github.com>.
> + <!-- identity endpoint -->
> + <test.rackspace-cloudfiles-us.endpoint>https://identity.api.rackspacecloud.com/v2.0/</test.rackspace-cloudfiles-us.endpoint>
> + <test.rackspace-cloudfiles-us.api-version>1</test.rackspace-cloudfiles-us.api-version>
> + <test.rackspace-cloudfiles-us.build-version />
> + <test.rackspace-cloudfiles-us.identity>${test.rackspace-us.identity}</test.rackspace-cloudfiles-us.identity>
> + <test.rackspace-cloudfiles-us.credential>${test.rackspace-us.credential}</test.rackspace-cloudfiles-us.credential>
> + <jclouds.osgi.export>org.jclouds.rackspace.cloudfiles.us*;version="${project.version}"</jclouds.osgi.export>
> + <jclouds.osgi.import>
> + org.jclouds.rest.internal;version="${jclouds.version}",
> + org.jclouds.labs*;version="${project.version}",
> + org.jclouds*;version="${jclouds.version}",
> + *
> + </jclouds.osgi.import>
> + </properties>
> +
> + <repositories>
probably no longer needed
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565/files#r18675266
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by BuildHive <no...@github.com>.
[jclouds » jclouds #1768](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/1768/) SUCCESS
This pull request looks good
[(what's this?)](https://www.cloudbees.com/what-is-buildhive)
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565#issuecomment-58597421
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-pull-requests-java-6 #184](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests-java-6/184/) SUCCESS
This pull request looks good
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565#issuecomment-58587305
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-pull-requests-java-6 #185](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests-java-6/185/) SUCCESS
This pull request looks good
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565#issuecomment-58587554
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-pull-requests #1272](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/1272/) SUCCESS
This pull request looks good
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565#issuecomment-58589523
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by BuildHive <no...@github.com>.
[jclouds » jclouds #1770](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/1770/) SUCCESS
This pull request looks good
[(what's this?)](https://www.cloudbees.com/what-is-buildhive)
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565#issuecomment-58604367
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by Adrian Cole <no...@github.com>.
> + http://www.apache.org/licenses/LICENSE-2.0
> +
> + Unless required by applicable law or agreed to in writing, software
> + distributed under the License is distributed on an "AS IS" BASIS,
> + WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
> + See the License for the specific language governing permissions and
> + limitations under the License.
> +
> +-->
> +<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
> + <modelVersion>4.0.0</modelVersion>
> + <parent>
> + <groupId>org.apache.jclouds</groupId>
> + <artifactId>jclouds-project</artifactId>
> + <version>2.0.0-SNAPSHOT</version>
> + <relativePath />
forgot to give an example for these..
```
<relativePath>../../project/pom.xml</relativePath>
```
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565/files#r18677093
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by Ignasi Barrera <no...@github.com>.
> + <id>apache-snapshots</id>
> + <url>https://repository.apache.org/content/repositories/snapshots</url>
> + <releases>
> + <enabled>false</enabled>
> + </releases>
> + <snapshots>
> + <enabled>true</enabled>
> + </snapshots>
> + </repository>
> + </repositories>
> +
> + <dependencies>
> + <dependency>
> + <groupId>org.apache.jclouds.api</groupId>
> + <artifactId>openstack-swift</artifactId>
> + <version>${project.parent.version}</version>
This is no longer needed. Use `${project.version}` everywhere.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565/files#r18675462
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by Adrian Cole <no...@github.com>.
and old cloudservers, too..
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565#issuecomment-58589683
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by Adrian Cole <no...@github.com>.
some pom cleanup is probably good. that and updating allblobstore/pom.xml
otherwise LG
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565#issuecomment-58580495
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by Adrian Cole <no...@github.com>.
> + http://www.apache.org/licenses/LICENSE-2.0
> +
> + Unless required by applicable law or agreed to in writing, software
> + distributed under the License is distributed on an "AS IS" BASIS,
> + WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
> + See the License for the specific language governing permissions and
> + limitations under the License.
> +
> +-->
> +<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
> + <modelVersion>4.0.0</modelVersion>
> + <parent>
> + <groupId>org.apache.jclouds</groupId>
> + <artifactId>jclouds-project</artifactId>
> + <version>2.0.0-SNAPSHOT</version>
> + <relativePath />
is this correct?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565/files#r18675324
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-pull-requests #1270](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/1270/) SUCCESS
This pull request looks good
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565#issuecomment-58583530
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by Adrian Cole <no...@github.com>.
> + <name>jclouds rackspace-cloudfiles api</name>
> + <description>jclouds components to access Rackspace Cloud Files</description>
> + <packaging>bundle</packaging>
> +
> + <properties>
> + <!-- identity endpoint -->
> + <test.rackspace-cloudfiles.endpoint>https://identity.api.rackspacecloud.com/v2.0/</test.rackspace-cloudfiles.endpoint>
> + <test.rackspace-cloudfiles.api-version>1</test.rackspace-cloudfiles.api-version>
> + <test.rackspace-cloudfiles.build-version />
> + <test.rackspace-cloudfiles.identity>FIXME_IDENTITY</test.rackspace-cloudfiles.identity>
> + <test.rackspace-cloudfiles.credential>FIXME_CREDENTIALS</test.rackspace-cloudfiles.credential>
> + <jclouds.osgi.export>org.jclouds.rackspace.cloudfiles.v1*;version="${project.version}"</jclouds.osgi.export>
> + <jclouds.osgi.import>org.jclouds*;version="${jclouds.version}",*</jclouds.osgi.import>
> + </properties>
> +
> + <repositories>
ditto on the repo stanza kill, in case forgot.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565/files#r18677112
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by Jeremy Daggett <no...@github.com>.
> + <name>jclouds rackspace-cloudfiles api</name>
> + <description>jclouds components to access Rackspace Cloud Files</description>
> + <packaging>bundle</packaging>
> +
> + <properties>
> + <!-- identity endpoint -->
> + <test.rackspace-cloudfiles.endpoint>https://identity.api.rackspacecloud.com/v2.0/</test.rackspace-cloudfiles.endpoint>
> + <test.rackspace-cloudfiles.api-version>1</test.rackspace-cloudfiles.api-version>
> + <test.rackspace-cloudfiles.build-version />
> + <test.rackspace-cloudfiles.identity>FIXME_IDENTITY</test.rackspace-cloudfiles.identity>
> + <test.rackspace-cloudfiles.credential>FIXME_CREDENTIALS</test.rackspace-cloudfiles.credential>
> + <jclouds.osgi.export>org.jclouds.rackspace.cloudfiles.v1*;version="${project.version}"</jclouds.osgi.export>
> + <jclouds.osgi.import>org.jclouds*;version="${jclouds.version}",*</jclouds.osgi.import>
> + </properties>
> +
> + <repositories>
Doh, hold on!
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565/files#r18678615
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by Adrian Cole <no...@github.com>.
@jdaggett one last find/replace, then LGTM
I'm in favor of removing the old providers, for sure. We'll probably need to keep swift for another release, but for the cloudfiles, I'd say do it!
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565#issuecomment-58589652
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by BuildHive <no...@github.com>.
[jclouds » jclouds #1766](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/1766/) SUCCESS
This pull request looks good
[(what's this?)](https://www.cloudbees.com/what-is-buildhive)
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565#issuecomment-58586702
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-pull-requests-java-6 #183](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests-java-6/183/) SUCCESS
This pull request looks good
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565#issuecomment-58586835
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by Ignasi Barrera <no...@github.com>.
> + See the License for the specific language governing permissions and
> + limitations under the License.
> +
> +-->
> +<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
> + <modelVersion>4.0.0</modelVersion>
> + <parent>
> + <groupId>org.apache.jclouds</groupId>
> + <artifactId>jclouds-project</artifactId>
> + <version>2.0.0-SNAPSHOT</version>
> + <relativePath />
> + </parent>
> +
> + <groupId>org.apache.jclouds.api</groupId>
> + <artifactId>rackspace-cloudfiles</artifactId>
> + <version>2.0.0-SNAPSHOT</version>
This will now always be released with the same version than the parent. I'd suggest removing the version from here to avoid mistakes.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565/files#r18675432
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by Adrian Cole <no...@github.com>.
> + See the License for the specific language governing permissions and
> + limitations under the License.
> +
> +-->
> +<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
> + <modelVersion>4.0.0</modelVersion>
> + <parent>
> + <groupId>org.apache.jclouds</groupId>
> + <artifactId>jclouds-project</artifactId>
> + <version>2.0.0-SNAPSHOT</version>
> + <relativePath />
> + </parent>
> +
> + <groupId>org.apache.jclouds.provider</groupId>
> + <artifactId>rackspace-cloudfiles-us</artifactId>
> + <version>2.0.0-SNAPSHOT</version>
probably don't need this
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565/files#r18675351
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by Jeremy Daggett <no...@github.com>.
> + <!-- identity endpoint -->
> + <test.rackspace-cloudfiles-us.endpoint>https://identity.api.rackspacecloud.com/v2.0/</test.rackspace-cloudfiles-us.endpoint>
> + <test.rackspace-cloudfiles-us.api-version>1</test.rackspace-cloudfiles-us.api-version>
> + <test.rackspace-cloudfiles-us.build-version />
> + <test.rackspace-cloudfiles-us.identity>${test.rackspace-us.identity}</test.rackspace-cloudfiles-us.identity>
> + <test.rackspace-cloudfiles-us.credential>${test.rackspace-us.credential}</test.rackspace-cloudfiles-us.credential>
> + <jclouds.osgi.export>org.jclouds.rackspace.cloudfiles.us*;version="${project.version}"</jclouds.osgi.export>
> + <jclouds.osgi.import>
> + org.jclouds.rest.internal;version="${jclouds.version}",
> + org.jclouds.labs*;version="${project.version}",
> + org.jclouds*;version="${jclouds.version}",
> + *
> + </jclouds.osgi.import>
> + </properties>
> +
> + <repositories>
Is this entire block defined elsewhere in a parent pom now?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565/files#r18675434
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by Adrian Cole <no...@github.com>.
ok just ping me and I'll give a last look when you're ready, @jdaggett. Thanks for doing this! This will be the first time we can allow users to switch off the old stuff, and not be in labs, since nova.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565#issuecomment-58584121
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by Adrian Cole <no...@github.com>.
> + <groupId>org.apache.jclouds.provider</groupId>
> + <artifactId>rackspace-cloudfiles-us</artifactId>
> + <version>2.0.0-SNAPSHOT</version>
> + <name>jclouds Rackspace Cloud Files US provider</name>
> + <description>OpenStack Object Storage implementation targeted to Rackspace Cloud Files US</description>
> + <packaging>bundle</packaging>
> +
> + <properties>
> + <!-- identity endpoint -->
> + <test.rackspace-cloudfiles-us.endpoint>https://identity.api.rackspacecloud.com/v2.0/</test.rackspace-cloudfiles-us.endpoint>
> + <test.rackspace-cloudfiles-us.api-version>1</test.rackspace-cloudfiles-us.api-version>
> + <test.rackspace-cloudfiles-us.build-version />
> + <test.rackspace-cloudfiles-us.identity>${test.rackspace-us.identity}</test.rackspace-cloudfiles-us.identity>
> + <test.rackspace-cloudfiles-us.credential>${test.rackspace-us.credential}</test.rackspace-cloudfiles-us.credential>
> + <jclouds.osgi.export>org.jclouds.rackspace.cloudfiles.us*;version="${project.version}"</jclouds.osgi.export>
> + <jclouds.osgi.import>
https://github.com/jclouds/jclouds/blob/master/providers/ultradns-ws/pom.xml#L42
ex.
```
<jclouds.osgi.import>org.jclouds*;version="${project.version}",*</jclouds.osgi.import>
```
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565/files#r18676355
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by Jeremy Daggett <no...@github.com>.
> + http://www.apache.org/licenses/LICENSE-2.0
> +
> + Unless required by applicable law or agreed to in writing, software
> + distributed under the License is distributed on an "AS IS" BASIS,
> + WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
> + See the License for the specific language governing permissions and
> + limitations under the License.
> +
> +-->
> +<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
> + <modelVersion>4.0.0</modelVersion>
> + <parent>
> + <groupId>org.apache.jclouds</groupId>
> + <artifactId>jclouds-project</artifactId>
> + <version>2.0.0-SNAPSHOT</version>
> + <relativePath />
Ah yes! The reason we had an empty relative path there was because Maven complained about the separate labs project.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565/files#r18678587
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-pull-requests-java-6 #182](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests-java-6/182/) SUCCESS
This pull request looks good
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565#issuecomment-58582469
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by Everett Toews <no...@github.com>.
Superseded by #705
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565#issuecomment-85716224
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by Adrian Cole <no...@github.com>.
> + <id>apache-snapshots</id>
> + <url>https://repository.apache.org/content/repositories/snapshots</url>
> + <releases>
> + <enabled>false</enabled>
> + </releases>
> + <snapshots>
> + <enabled>true</enabled>
> + </snapshots>
> + </repository>
> + </repositories>
> +
> + <dependencies>
> + <dependency>
> + <groupId>org.apache.jclouds.api</groupId>
> + <artifactId>openstack-swift</artifactId>
> + <version>${project.parent.version}</version>
switch to parent.version
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565/files#r18675376
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by Jeremy Daggett <no...@github.com>.
@demobox Just need to backport the changes to 1.8.x once I get the :+1: from you. Thx!
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565#issuecomment-58580133
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by Ignasi Barrera <no...@github.com>.
It seems that we've both started to review and comment at the same time @adriancole :) Just to avoid repeating comments, all yours!
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565#issuecomment-58581052
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-pull-requests #1271](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/1271/) SUCCESS
This pull request looks good
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565#issuecomment-58585409
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by BuildHive <no...@github.com>.
[jclouds » jclouds #1769](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/1769/) SUCCESS
This pull request looks good
[(what's this?)](https://www.cloudbees.com/what-is-buildhive)
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565#issuecomment-58600966
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by Jeremy Daggett <no...@github.com>.
@adriancole Looks like this is good to go. I am going to submit a PR for 1.8.x in just a min here. WDYT about removing the old API/providers?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565#issuecomment-58589040
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by BuildHive <no...@github.com>.
[jclouds » jclouds #1767](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/1767/) SUCCESS
This pull request looks good
[(what's this?)](https://www.cloudbees.com/what-is-buildhive)
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565#issuecomment-58592736
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by Ignasi Barrera <no...@github.com>.
> + <!-- identity endpoint -->
> + <test.rackspace-cloudfiles-us.endpoint>https://identity.api.rackspacecloud.com/v2.0/</test.rackspace-cloudfiles-us.endpoint>
> + <test.rackspace-cloudfiles-us.api-version>1</test.rackspace-cloudfiles-us.api-version>
> + <test.rackspace-cloudfiles-us.build-version />
> + <test.rackspace-cloudfiles-us.identity>${test.rackspace-us.identity}</test.rackspace-cloudfiles-us.identity>
> + <test.rackspace-cloudfiles-us.credential>${test.rackspace-us.credential}</test.rackspace-cloudfiles-us.credential>
> + <jclouds.osgi.export>org.jclouds.rackspace.cloudfiles.us*;version="${project.version}"</jclouds.osgi.export>
> + <jclouds.osgi.import>
> + org.jclouds.rest.internal;version="${jclouds.version}",
> + org.jclouds.labs*;version="${project.version}",
> + org.jclouds*;version="${jclouds.version}",
> + *
> + </jclouds.osgi.import>
> + </properties>
> +
> + <repositories>
Nope but the build itself will already build all the needed SNAPSHOT dependencies. The repo is needed in projects in other Git repos where the dependencies are not part of the Maven build.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565/files#r18675532
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-pull-requests #1274](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/1274/) SUCCESS
This pull request looks good
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565#issuecomment-58590151
Re: [jclouds] Import rackspace-cloudfiles from labs (#565)
Posted by Adrian Cole <no...@github.com>.
> + <id>apache-snapshots</id>
> + <url>https://repository.apache.org/content/repositories/snapshots</url>
> + <releases>
> + <enabled>false</enabled>
> + </releases>
> + <snapshots>
> + <enabled>true</enabled>
> + </snapshots>
> + </repository>
> + </repositories>
> +
> + <dependencies>
> + <dependency>
> + <groupId>org.apache.jclouds.api</groupId>
> + <artifactId>openstack-swift</artifactId>
> + <version>${project.parent.version}</version>
@jdaggett I'd do a find/replace sweep on the poms.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/565/files#r18679418