[
https://issues.apache.org/jira/browse/LIBCLOUD-266?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13529623#comment-13529623
]
Tomaz Muraus commented on LIBCLOUD-266:
---------------------------------------
First thanks for the patch again!
I did have a look at it a while ago when you initially submitted it and there
were some issues I have noticed:
1. Classes aren't old style (they don't inherit from object)
2. "raise Exception" is also deprecated, you should do raise
Exception('message') or in most cases throw a more specific exception
3. this patch actually doesn't work by itself because gcelib depends on oauth
library which is not installed
I'm not yet sure what's the best way to proceed with regards to #3. We don't
want to add any more dependencies to Libcloud for the reasons already discussed
in the past.
I'm leaning towards not using gcelib again. I've checked the GCE API docs and
there is nothing to insane in it so it wouldn't be that hard to implement it
from scratch in Libcloud. Only part which would take a bit more time is
authentication (oauth2).
> Adding Google Compute Engine Support
> ------------------------------------
>
> Key: LIBCLOUD-266
> URL: https://issues.apache.org/jira/browse/LIBCLOUD-266
> Project: Libcloud
> Issue Type: Improvement
> Components: Compute
> Affects Versions: 0.11.2
> Reporter: Ziyad Mir
> Priority: Minor
> Labels: patch
> Attachments: libcloud.patch
>
>
> Adding a new compute driver for Google Compute Engine.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
