You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@jclouds.apache.org by Andrew Gaul <no...@github.com> on 2015/05/27 03:13:58 UTC

[jclouds-karaf] JCLOUDS-669: Add support for openstack-swift and rackspace-cloudfiles (#67)

You can view, comment on, or merge this pull request online at:

  https://github.com/jclouds/jclouds-karaf/pull/67

-- Commit Summary --

  * JCLOUDS-669: Add support for openstack-swift
  * JCLOUDS-669: Add support for rackspace-cloudfiles

-- File Changes --

    M feature/pom.xml (5)
    M feature/src/main/resources/feature.xml (20)

-- Patch Links --

https://github.com/jclouds/jclouds-karaf/pull/67.patch
https://github.com/jclouds/jclouds-karaf/pull/67.diff

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-karaf/pull/67

Re: [jclouds-karaf] JCLOUDS-669: Add support for openstack-swift and rackspace-cloudfiles (#67)

Posted by Zack Shoylev <no...@github.com>.
Why not just remove the old providers?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-karaf/pull/67#issuecomment-105965118

Re: [jclouds-karaf] JCLOUDS-669: Add support for openstack-swift and rackspace-cloudfiles (#67)

Posted by Andrew Gaul <no...@github.com>.
@zack-shoylev @iocanel please review

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-karaf/pull/67#issuecomment-105708993

Re: [jclouds-karaf] JCLOUDS-669: Add support for openstack-swift and rackspace-cloudfiles (#67)

Posted by Andrew Gaul <no...@github.com>.
I amended the commits to add the suggested tests.  We can remove the legacy support in a subsequent pull request.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-karaf/pull/67#issuecomment-106059667

Re: [jclouds-karaf] JCLOUDS-669: Add support for openstack-swift and rackspace-cloudfiles (#67)

Posted by Andrew Gaul <no...@github.com>.
Needed cloudidentity dependency.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-karaf/pull/67#issuecomment-106096183

Re: [jclouds-karaf] JCLOUDS-669: Add support for openstack-swift and rackspace-cloudfiles (#67)

Posted by Ignasi Barrera <no...@github.com>.
Add the corresponding tests to [ApisFeaturesInstallationTest](https://github.com/jclouds/jclouds-karaf/blob/master/itests/src/test/java/org/jclouds/karaf/itests/ApisFeaturesInstallationTest.java) and [RackspaceFeaturesInstallationTest](https://github.com/jclouds/jclouds-karaf/blob/master/itests/src/test/java/org/jclouds/karaf/itests/RackspaceFeaturesInstallationTest.java)

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-karaf/pull/67#issuecomment-105828748

Re: [jclouds-karaf] JCLOUDS-669: Add support for openstack-swift and rackspace-cloudfiles (#67)

Posted by Andrew Gaul <no...@github.com>.
Odd, not sure why the openstack-swift tests succeed but the rackspace-cloudfiles tests fail?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-karaf/pull/67#issuecomment-106065400