You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@jclouds.apache.org by Ignasi Barrera <no...@github.com> on 2014/03/27 11:43:25 UTC

[jclouds] Update ElasticHosts pre-installed images and added new regions (#331)

https://issues.apache.org/jira/browse/JCLOUDS-517

* Added new ElasticHosts regions: Amsterdam, San José, Hong Kong and Sydney
* Fixed the pre-installed image list according to: http://www.elastichosts.com/support/api/ (see the &quot;Standard drives&quot; section)
* Fixed a minor issue in the ComputeService: before starting to image a drive, we have to wait until it is not claimed.
You can merge this Pull Request by running:

  git pull https://github.com/nacx/jclouds elastichosts

Or you can view, comment on it, or merge it online at:

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

-- Commit Summary --

  * New ElasticHosts images and regions

-- File Changes --

    M apis/elasticstack/src/main/java/org/jclouds/elasticstack/compute/ElasticStackComputeServiceAdapter.java (16)
    A providers/elastichosts-ams-e/README.txt (6)
    A providers/elastichosts-ams-e/pom.xml (127)
    A providers/elastichosts-ams-e/src/main/java/org/jclouds/elastichosts/ElasticHostsAmsterdamMetadata.java (83)
    A providers/elastichosts-ams-e/src/main/resources/META-INF/services/org.jclouds.providers.ProviderMetadata (1)
    A providers/elastichosts-ams-e/src/main/resources/elastichosts-ams-e/preinstalled_images.json (79)
    A providers/elastichosts-ams-e/src/test/java/org/jclouds/elastichosts/ElasticHostsAmsterdamApiLiveTest.java (30)
    A providers/elastichosts-ams-e/src/test/java/org/jclouds/elastichosts/ElasticHostsAmsterdamProviderTest.java (34)
    A providers/elastichosts-ams-e/src/test/java/org/jclouds/elastichosts/compute/ElasticHostsAmsterdamComputeServiceLiveTest.java (33)
    A providers/elastichosts-ams-e/src/test/java/org/jclouds/elastichosts/compute/ElasticHostsAmsterdamTemplateBuilderLiveTest.java (83)
    A providers/elastichosts-hkg-e/README.txt (5)
    A providers/elastichosts-hkg-e/pom.xml (127)
    A providers/elastichosts-hkg-e/src/main/java/org/jclouds/elastichosts/ElasticHostsHongKongProviderMetadata.java (83)
    A providers/elastichosts-hkg-e/src/main/resources/META-INF/services/org.jclouds.providers.ProviderMetadata (1)
    A providers/elastichosts-hkg-e/src/main/resources/elastichosts-hkg-e/preinstalled_images.json (79)
    A providers/elastichosts-hkg-e/src/test/java/org/jclouds/elastichosts/ElasticHostsHongKongApiLiveTest.java (30)
    A providers/elastichosts-hkg-e/src/test/java/org/jclouds/elastichosts/ElasticHostsHongKongProviderTest.java (32)
    A providers/elastichosts-hkg-e/src/test/java/org/jclouds/elastichosts/compute/ElasticHostsHongKongComputeServiceLiveTest.java (33)
    A providers/elastichosts-hkg-e/src/test/java/org/jclouds/elastichosts/compute/ElasticHostsHongKongTemplateBuilderLiveTest.java (83)
    M providers/elastichosts-lax-p/src/main/resources/elastichosts-lax-p/preinstalled_images.json (56)
    M providers/elastichosts-lax-p/src/test/java/org/jclouds/elastichosts/compute/ElasticHostsPeer1LosAngelesTemplateBuilderLiveTest.java (13)
    M providers/elastichosts-lon-b/src/main/resources/elastichosts-lon-b/preinstalled_images.json (58)
    M providers/elastichosts-lon-b/src/test/java/org/jclouds/elastichosts/compute/ElasticHostsBlueSquareLondonTemplateBuilderLiveTest.java (18)
    M providers/elastichosts-lon-p/src/main/resources/elastichosts-lon-p/preinstalled_images.json (56)
    M providers/elastichosts-lon-p/src/test/java/org/jclouds/elastichosts/compute/ElasticHostsPeer1LondonTemplateBuilderLiveTest.java (12)
    M providers/elastichosts-sat-p/src/main/resources/elastichosts-sat-p/preinstalled_images.json (61)
    M providers/elastichosts-sat-p/src/test/java/org/jclouds/elastichosts/compute/ElasticHostsPeer1SanAntonioTemplateBuilderLiveTest.java (13)
    A providers/elastichosts-sjc-c/README.txt (5)
    A providers/elastichosts-sjc-c/pom.xml (127)
    A providers/elastichosts-sjc-c/src/main/java/org/jclouds/elastichosts/ElasticHostsSanJoseProviderMetadata.java (83)
    A providers/elastichosts-sjc-c/src/main/resources/META-INF/services/org.jclouds.providers.ProviderMetadata (1)
    A providers/elastichosts-sjc-c/src/main/resources/elastichosts-sjc-c/preinstalled_images.json (79)
    A providers/elastichosts-sjc-c/src/test/java/org/jclouds/elastichosts/ElasticHostsSanJoseApiLiveTest.java (30)
    A providers/elastichosts-sjc-c/src/test/java/org/jclouds/elastichosts/ElasticHostsSanJoseProviderTest.java (32)
    A providers/elastichosts-sjc-c/src/test/java/org/jclouds/elastichosts/compute/ElasticHostsSanJoseComputeServiceLiveTest.java (33)
    A providers/elastichosts-sjc-c/src/test/java/org/jclouds/elastichosts/compute/ElasticHostsSanJoseTemplateBuilderLiveTest.java (83)
    A providers/elastichosts-syd-v/README.txt (5)
    A providers/elastichosts-syd-v/pom.xml (127)
    A providers/elastichosts-syd-v/src/main/java/org/jclouds/elastichosts/ElasticHostsSydneyProviderMetadata.java (83)
    A providers/elastichosts-syd-v/src/main/resources/META-INF/services/org.jclouds.providers.ProviderMetadata (1)
    A providers/elastichosts-syd-v/src/main/resources/elastichosts-syd-v/preinstalled_images.json (79)
    A providers/elastichosts-syd-v/src/test/java/org/jclouds/elastichosts/ElasticHostsSydneyApiLiveTest.java (30)
    A providers/elastichosts-syd-v/src/test/java/org/jclouds/elastichosts/ElasticHostsSydneyProviderTest.java (32)
    A providers/elastichosts-syd-v/src/test/java/org/jclouds/elastichosts/compute/ElasticHostsSydneyComputeServiceLiveTest.java (33)
    A providers/elastichosts-syd-v/src/test/java/org/jclouds/elastichosts/compute/ElasticHostsSydneyTemplateBuilderLiveTest.java (83)
    M providers/elastichosts-tor-p/src/main/resources/elastichosts-tor-p/preinstalled_images.json (56)
    M providers/elastichosts-tor-p/src/test/java/org/jclouds/elastichosts/compute/ElasticHostsPeer1TorontoTemplateBuilderLiveTest.java (13)
    M providers/pom.xml (4)

-- Patch Links --

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

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

Re: [jclouds] Update ElasticHosts pre-installed images and added new regions (#331)

Posted by Audrius Butkevicius <no...@github.com>.
These are part of ElasticStack API, though requires a certain version of the software.

I believe that only one service provider has a version old enough not to have these API calls.

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

Re: [jclouds] Update ElasticHosts pre-installed images and added new regions (#331)

Posted by Ignasi Barrera <no...@github.com>.
GitHub does not seem to properly detect the rebased commits after reopening. Will open a new pull request.

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

Re: [jclouds] Update ElasticHosts pre-installed images and added new regions (#331)

Posted by Ignasi Barrera <no...@github.com>.
Will reopen as soon as I've reworked the fix to include the new API methods as said here.

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

Re: [jclouds] Update ElasticHosts pre-installed images and added new regions (#331)

Posted by BuildHive <no...@github.com>.
[jclouds » jclouds #970](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/970/) 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/331#issuecomment-38812095

Re: [jclouds] Update ElasticHosts pre-installed images and added new regions (#331)

Posted by Andrew Phillips <no...@github.com>.
@nacx: Status of this? Are we still trying to get this in for 1.7.2?

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

Re: [jclouds] Update ElasticHosts pre-installed images and added new regions (#331)

Posted by Audrius Butkevicius <no...@github.com>.
Hi,

Sorry for joining the party a bit late, but there now a few yet undocumented API calls to retrieve the standard images:

`/drives/list/standard` - To get a list of UUIDs for our standard images.
`/drives/info/standard` - To get information about all the standard images.

You can also use the standard image's UUID with normal drive calls:
`/drive/XYZ/info` - Retrieves basic information about drive XYZ.
`/drive/XYZ/info/full` - Retrieves detailed information about drive XYZ.


Regards,
Audrius.

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

Re: [jclouds] Update ElasticHosts pre-installed images and added new regions (#331)

Posted by Audrius Butkevicius <no...@github.com>.
All of them, apart from Serverlove.

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

Re: [jclouds] Update ElasticHosts pre-installed images and added new regions (#331)

Posted by Ignasi Barrera <no...@github.com>.
The supported providers that use the ElasticStack API are:

* ElasticHosts
* Open Hosting East1
* Serverlove Manchester
* SkaliCloud Malaysia
* Go2Cloud Johannesburg1

Do you know which of those providers support the new calls?

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

Re: [jclouds] Update ElasticHosts pre-installed images and added new regions (#331)

Posted by Ignasi Barrera <no...@github.com>.
Closed #331.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/331#event-120162125

Re: [jclouds] Update ElasticHosts pre-installed images and added new regions (#331)

Posted by Ignasi Barrera <no...@github.com>.
I'd definitely like to have it, otherwise the elastichosts providers can be considered broken.

I'm almost done with the changes. Just have to finish some cleanup and run the live tests, hope to have it in a couple days. Adding the review cycles, it would be completed by the end of the week. Don't know if the release can wait till then.

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

Re: [jclouds] Update ElasticHosts pre-installed images and added new regions (#331)

Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-java-7-pull-requests #1184](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/1184/) SUCCESS
This pull request looks good

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

Re: [jclouds] Update ElasticHosts pre-installed images and added new regions (#331)

Posted by BuildHive <no...@github.com>.
[jclouds » jclouds #1085](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/1085/) 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/331#issuecomment-42859414

Re: [jclouds] Update ElasticHosts pre-installed images and added new regions (#331)

Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-java-7-pull-requests #1265](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/1265/) SUCCESS
This pull request looks good

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

Re: [jclouds] Update ElasticHosts pre-installed images and added new regions (#331)

Posted by Ignasi Barrera <no...@github.com>.
Thanks for the feedback @AudriusButkevicius.

Are these api calls part of the ElasticStack API or are them ElasticHosts specific? There are other providers that use the ElasticStack API, and we need to know where these new API calls belong.

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

Re: [jclouds] Update ElasticHosts pre-installed images and added new regions (#331)

Posted by BuildHive <no...@github.com>.
[jclouds » jclouds #1084](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/1084/) 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/331#issuecomment-42852940

Re: [jclouds] Update ElasticHosts pre-installed images and added new regions (#331)

Posted by Ignasi Barrera <no...@github.com>.
Ok, I'm going to add the calls to the ElasticStack API then and keep the hardcoded stuff for ServerLove. Do you know which version of ElasticStack introduces those calls, so I can properly document them?

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

Re: [jclouds] Update ElasticHosts pre-installed images and added new regions (#331)

Posted by Ignasi Barrera <no...@github.com>.
Closed #331.

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

Re: [jclouds] Update ElasticHosts pre-installed images and added new regions (#331)

Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-pull-requests #714](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/714/) SUCCESS
This pull request looks good

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

Re: [jclouds] Update ElasticHosts pre-installed images and added new regions (#331)

Posted by Audrius Butkevicius <no...@github.com>.
So the API method has been there for over a year now.

Currently there isn't a way to identify the API version externally, therefore even if I were to provide an internal version number, there is no way for checking it.

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

Re: [jclouds] Update ElasticHosts pre-installed images and added new regions (#331)

Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-pull-requests #795](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/795/) SUCCESS
This pull request looks good

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