Thanks for starting this PR @sivikt! Having a look over the PR, there are a lot 
of TODOs to be done before we can merge it. Basically: 

* The error handlers must be configured in the rest module.
* If the computeservice is not implemented I'd just remove it and leave only 
the Server API. The compute service adapter can be added later, once the api is 
complete and works.
* There are two empty apis. I'd remove and re-add them once they have some 
method implemented.
* Expect tests and live tests are mandatory for us. We need them to be 
confident that what we merge actually works.

Instead of having a pull request with a "skeleton" of the entire provider, I 
think it would be better to go step by step, and have complete features, one by 
one. This way it will be easier to iterate and have the provider properly 
implemented. Sound good to you?

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

Reply via email to