<div dir="ltr">Hi Ghe,<div><br></div><div style>My first thought aligns with yours--this code should be moved into glanceclient rather than oslo.</div><div style><br></div><div style>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 style><br></div><div style>Partially just thinking out loud. . .</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">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>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><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 class="HOEnZb"><font color="#888888"><br>
<br>
</font></span></div><span class="HOEnZb"><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>_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">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></blockquote></div><br></div>