Open Stack

Hi Mark and all,
    as you pointed out, the idea is to get rid of all the wrappers in
nova/cinder and move the features onto glanceclient. Glanceclient is more
http oriented, while these wrappers help you to use glanceclient on an
upper level (using a context for connection auth, you get a dict for the
image metadata, instead of a json on a http header...).

So my plan is:
1/ move those features to glanceclient,
2/ Use this new glanceclient on ironic as a proof of concept.
3/ Remove the wrappers on nova and cinder and use glanceclient directly
4/ Start working on glanceclient for new features, improvements once the
base code is the same on all the projects.

Being this a big change, although most of the code should be the same, time
is important, so my idea is to have 1 and 2 on a couple of weeks, and 3
before havana-2.


On Sat, Jun 1, 2013 at 7:08 PM, Mark Washenberger <
mark.washenberger at markwash.net> wrote:

> Hi Ghe,
>
> My first thought aligns with yours--this code should be moved into
> glanceclient rather than oslo.
>
> However, after digging a bit deeper I'd love to find out why this code
> even needs to exist in the first place. It seems like every reason it
> exists is actually just a deficiency in the feature-set of the glanceclient
> library that already exists. So instead of porting this code to glance,
> would it be possible to port its features to existing code in glance
> client, and use glanceclient directly instead of wrapper code in nova,
> cinder, and ironic?
>
> Partially just thinking out loud. . .
>
>
> On Sat, Jun 1, 2013 at 2:22 AM, Ghe Rivero <ghe at debian.org> wrote:
>
>> Hi,
>>     as part of the new Ironic project, one of out needs is to retrieve
>> images from glance for later deployment on the baremetal nodes. Until now,
>> we were using the implementation of the service inside nova (which also has
>> an almost exact copy on cinder) located on:
>>
>> nova.virt.images ~ cinder.image.image_utils
>>
>> nova.image.glance ~ cinder.image.glance
>> with a few external dependencies (mostly some exceptions and
>> nova.utils/cinder.utils tools.)
>> https://blueprints.launchpad.net/ironic/+spec/image-tools
>>
>> My first idea was to move this common service (needed now by 3 projects)
>> into oslo (https://review.openstack.org/#/q/topic:bp/image-tools,n,z),
>> but in a second thought, glanceclient looks like a better candidate for it.
>>
>> I would to know your opinions and concerns about this. My only one is the
>> huge patch for the openstack-common modules needed (+9000 lines), but
>> pointing it out ahead on tie, should be pretty easy to solve.
>>
>> Ghe Rivero
>>
>> --
>> Pinky: "Gee, Brain, what do you want to do tonight?"
>> The Brain: "The same thing we do every night, Pinky—try to take over the
>> world!"
>>
>>  .''`.  Pienso, Luego Incordio
>> : :' :
>> `. `'
>>   `-    www.debian.org    www.openstack.com
>>
>> GPG Key: 26F020F7
>> GPG fingerprint: 4986 39DA D152 050B 4699  9A71 66DB 5A36 26F0 20F7
>>
>> _______________________________________________
>> OpenStack-dev mailing list
>> OpenStack-dev at lists.openstack.org
>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>>
>>
>


-- 
Pinky: "Gee, Brain, what do you want to do tonight?"
The Brain: "The same thing we do every night, Pinky—try to take over the
world!"

 .''`.  Pienso, Luego Incordio
: :' :
`. `'
  `-    www.debian.org    www.openstack.com

GPG Key: 26F020F7
GPG fingerprint: 4986 39DA D152 050B 4699  9A71 66DB 5A36 26F0 20F7
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20130602/d7ee55d6/attachment.html>