Yes, I just modified it in my branch here: https://code.launchpad.net/~vishvananda/nova/kill-objectstore/+merge/52163
So that we have an image service for single machine installs that doesn't depend on glance. I think we should keep it in for now. Vish On Mar 7, 2011, at 6:55 AM, Dan Prince wrote: > Is anyone using the nova/image/local.py? > > This class appears to be a pre-glance image service that could be used with > the Openstack API. As we now have glance I'm not sure we really need this > anymore. > > The class initializes its path to a temp directory: > > def __init__(self): > self._path = tempfile.mkdtemp() > > Maybe we can use this as a proper mock image service (within the tests) and > remove it as a top level image service within the code base. > > -- > I file a bug on this issue here: > > https://bugs.launchpad.net/nova/+bug/723947 > > NOTE: This has nothing to do with 'nova-objectstore' or anything on the > S3/EC2 side of things as some of the comments mention in the ticket. > -- > > Why remove a class that isn't hurting anything? In another bug/ticket I was > trying to standardize how the image services provide access to 'kernel_id' > and 'ramdisk_id' in a standardized manner. This will help clean up some of > the code in the API layers. While doing this I ran across nova/image/local.py > which seemed to be rather outdated. > > Dan > > > _______________________________________________ > Mailing list: https://launchpad.net/~openstack > Post to : [email protected] > Unsubscribe : https://launchpad.net/~openstack > More help : https://help.launchpad.net/ListHelp
_______________________________________________ Mailing list: https://launchpad.net/~openstack Post to : [email protected] Unsubscribe : https://launchpad.net/~openstack More help : https://help.launchpad.net/ListHelp
