Code review comment for lp://qastaging/~yamahata/glance/lp802893

Revision history for this message
Isaku Yamahata (yamahata) wrote :

On Tue, Jun 28, 2011 at 02:48:38PM -0000, Brian Waldon wrote:
> This is an interesting idea, but I'm a little hesitant to call this a bug. Keeping metadata simple was an explicit design design. I think this should probably be filed as a blueprint targeted at diablo-3, assuming this is something we want. Thoughts, Jay?

Fair enough, I'll file a blueprint and let's start discussion.
I think the discussion involves both nova and glance. So I'll also start
a discussion thread on openstack devel ml in order to draw attention from
nova developer. So far I've thought it's Glance issue so that I created
this patch to Glance. Other option would be to change nova metadata handling.

Jay, any comments?

thanks,

> I would also like to see a lot more testing around this. I would want to guarantee this will work at every level, not just the utils function that does the mapping.
--
yamahata

« Back to merge proposal