[openstack-dev] [NOVA] Removal of _get_image_metadata from nova/compute/api.py
Andrew Laski
andrew.laski at rackspace.com
Thu Jun 6 14:03:19 UTC 2013
On 06/06/13 at 12:07pm, Macdonald-Wallace, Matthew wrote:
>Hi all,
>
>As part of trying to get https://review.openstack.org/#/c/28969 through the review process, I have hit an issue with rebasing.
>
>The main issue appears to be that the function that all my code ran within (_get_image_metadata) was removed in change 6b16c8731c44e4a6c80b803f3e8afdd88386d577.
If it was removed, it occurred before this change. But just keep the
discussion close to the code in question I have commented on the review
you linked above.
>
>I'm now trying to understand where the Metadata from an image is being collected and passed through as it seems that now the get_image_metadata call has gone, the appropriate place to put my code (which has now been heavily refactored from the latest changeset in the above link!) would be in the get_image function (api.py:528).
>
>If get_image is indeed the correct place to put it, should this be renamed to something else to reflect that this is getting both volume and image data, or should I add a get_volume function with my code in it and then refactor the create and recreate functions to include this new function.
>
>Thanks in advance for any thoughts,
>
>Matt
>_______________________________________________
>OpenStack-dev mailing list
>OpenStack-dev at lists.openstack.org
>http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
More information about the OpenStack-dev
mailing list