[
https://issues.apache.org/jira/browse/LIBCLOUD-786?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15083976#comment-15083976
]
ASF subversion and git services commented on LIBCLOUD-786:
----------------------------------------------------------
Commit 81a8dbc324f870444e980aee523fea48fc37cbcc in libcloud's branch
refs/heads/trunk from [~crunk1]
[ https://git-wip-us.apache.org/repos/asf?p=libcloud.git;h=81a8dbc ]
Implemented LazyObject class, which provides a .lazy class method. The lazy
class method returns a Proxy object that subclasses the target object class.
Upon accessing the proxy object in any way, the object is initialized.
Modified Google Compute Engine License objects, GCELicense, to be such a lazy
object. This addresses https://issues.apache.org/jira/browse/LIBCLOUD-786.
Tests/Verification:
tox -e lint
python setup.py test
Added test/common/test_base.py which has LazyObjectTest
Signed-off-by: Eric Johnson <[email protected]>
> GCENodeDriver.ex_get_image needs some love, see description.
> ------------------------------------------------------------
>
> Key: LIBCLOUD-786
> URL: https://issues.apache.org/jira/browse/LIBCLOUD-786
> Project: Libcloud
> Issue Type: Bug
> Components: Compute
> Reporter: Scott Crunkleton
>
> Firstly, the parameter 'ex_project_list' is a misnomer. It is being treated
> as a single project, not list of projects.
> Secondly, it's slow. When not providing a project parameter, ex_get_image
> lists all images from all projects listed in GCENodeDriver.IMAGE_PROJECTS.
> This is a separate API call for each project. After getting all the images,
> any images that have licenses create another API call via ex_get_license.
> Thirdly, if any of the license calls try to pull a license from an image
> project that the user doesn't have access to, it raises a
> ResourceNotFoundError.
> My opinion: fix the ex_project_list param, and see about not populating the
> image license data when fetching images.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
