<div dir="ltr"><div class=""><img class="" id=":11l" tabindex="0" src="https://mail.google.com/mail/u/0/images/cleardot.gif" alt=""></div><div>Hi Mark and all,<br>
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...). <br>
<br></div><div>So my plan is: <br>1/ move those features to glanceclient, <br></div><div>2/ Use this new glanceclient on ironic as a proof of concept.<br></div><div>3/ Remove the wrappers on nova and cinder and use glanceclient directly<br>
</div><div>4/ Start working on glanceclient for new features, improvements once the base code is the same on all the projects.<br><br></div>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.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Jun 1, 2013 at 7:08 PM, Mark Washenberger <span dir="ltr"><<a href="mailto:mark.washenberger@markwash.net" target="_blank">mark.washenberger@markwash.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Ghe,<div><br></div><div>My first thought aligns with yours--this code should be moved into glanceclient rather than oslo.</div>
<div><br></div><div>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?</div>
<div><br></div><div>Partially just thinking out loud. . .</div></div><div class="gmail_extra"><br><br><div class="gmail_quote"><div><div class="h5">On Sat, Jun 1, 2013 at 2:22 AM, Ghe Rivero <span dir="ltr"><<a href="mailto:ghe@debian.org" target="_blank">ghe@debian.org</a>></span> wrote:<br>
</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5"><div dir="ltr"><div><div><div><div>Hi,<br></div> 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:<br>
<br>nova.virt.images ~ cinder.image.image_utils<br><p>
nova.image.glance ~ cinder.image.glance</p>with a few external dependencies (mostly some exceptions and nova.utils/cinder.utils tools.)<br><a href="https://blueprints.launchpad.net/ironic/+spec/image-tools" target="_blank">https://blueprints.launchpad.net/ironic/+spec/image-tools</a><br>
<br></div>My first idea was to move this common service (needed now by 3 projects) into oslo (<a href="https://review.openstack.org/#/q/topic:bp/image-tools,n,z" target="_blank">https://review.openstack.org/#/q/topic:bp/image-tools,n,z</a>), but in a second thought, glanceclient looks like a better candidate for it.<br>
<br></div>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.<span><font color="#888888"><br>
<br>
</font></span></div><span><font color="#888888">Ghe Rivero<br clear="all"><div><div><div><div><div><div><div><div><br>-- <br>Pinky: "Gee, Brain, what do you want to do tonight?"<br>The Brain: "The same thing we do every night, Pinky—try to take over the world!"<br>
<br> .''`. Pienso, Luego Incordio <br>: :' : <br>`. `' <br> `- <a href="http://www.debian.org" target="_blank">www.debian.org</a> <a href="http://www.openstack.com" target="_blank">www.openstack.com</a><br>
<br>GPG Key: 26F020F7<br>GPG fingerprint: 4986 39DA D152 050B 4699 9A71 66DB 5A36 26F0 20F7
</div></div></div></div></div></div></div></div></font></span></div>
<br></div></div><div class="im">_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org" target="_blank">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br></div></blockquote></div><br></div>
</blockquote></div><br><br clear="all"><br>-- <br>Pinky: "Gee, Brain, what do you want to do tonight?"<br>The Brain: "The same thing we do every night, Pinky—try to take over the world!"<br><br> .''`. Pienso, Luego Incordio <br>
: :' : <br>`. `' <br> `- <a href="http://www.debian.org" target="_blank">www.debian.org</a> <a href="http://www.openstack.com" target="_blank">www.openstack.com</a><br><br>GPG Key: 26F020F7<br>GPG fingerprint: 4986 39DA D152 050B 4699 9A71 66DB 5A36 26F0 20F7
</div>