<div dir="ltr">Flavio,<div><br></div><div><span style="font-family:arial,sans-serif;font-size:13px">Agreed. I'd also like to see other project migrated before pulling</span><br style="font-family:arial,sans-serif;font-size:13px">
<span style="font-family:arial,sans-serif;font-size:13px">oslo.db out from oslo-incubator</span></div><div><font face="arial, sans-serif"><br></font><div><br></div><div>as I wrote before oslo.db code is used by: Nova, Neutron, Cinder, Ironic, Ceilometer use oslo.db. And we have already patches to switch in Glance to id. And we are woking in Keystone and Heat. </div>
<div><br></div><div>Why (nova, neutron, cinder, ironic and ceilometer) is not enough to say that the code is OK?</div><div><br></div><div><br></div><div><span style="font-family:arial,sans-serif;font-size:13px">I'd also add that pulling oslo db into its own package means that</span><br style="font-family:arial,sans-serif;font-size:13px">
<span style="font-family:arial,sans-serif;font-size:13px">projects using Oslo's db code have to be migrated as well. I think</span><br style="font-family:arial,sans-serif;font-size:13px"><span style="font-family:arial,sans-serif;font-size:13px">it's a bit late for that. The focus should be on making sure current</span><br style="font-family:arial,sans-serif;font-size:13px">
<span style="font-family:arial,sans-serif;font-size:13px">code is stable enough for the not-so-far release.</span><br></div><div><br></div><div><br></div><div>Hm, I really don't see big problems with migrating to oslo.db lib even in this moment, because in oslo-incubator and in oslo.db is the same code.. Could you explain what problem you see in process of migartion? (For me it is add one more requirments, remove openstack/db/.. folder, switch imports)..</div>
<div><br></div><div><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Aug 19, 2013 at 1:29 PM, Flavio Percoco <span dir="ltr"><<a href="mailto:flavio@redhat.com" target="_blank">flavio@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div>On 19/08/13 00:34 -0700, Gary Kotton wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
Hi,<br>
<br>
I have a number of things to say here:<br>
<br>
1. Great work in getting the DB into the common and ironing out the<br>
issues<br>
<br>
</blockquote>
<br></div>
+1<div><br>
<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
2. As far as I know only Neutron and Nova are making use of the common DB<br>
code. Neutron has been using this since the beginning of H2 (this did not<br>
resolve all of the issues that we had and we had) and Nova has just recently<br>
upgraded to the latest DB code (this was a few weeks ago).<br>
<br>
3. In general I like the idea of having a separate lib for this but have<br>
a number of reservations regarding the timing and stability:<br>
<br>
a. I do not think that this has been running long enough in Neutron and<br>
Nova for us to give it a stamp of approval (the common CFG code was at least<br>
one cycle as common code prior to moving into its own lib). I think that in<br>
Neutron we still have a number of issues with load on the DB. I need to double<br>
check on this.<br>
</blockquote>
<br></div>
Agreed. I'd also like to see other project migrated before pulling<br>
oslo.db out from oslo-incubator<div><br>
<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
b. I think that the beginning of Icehouse is a good time. When we moved to<br>
the CFG library there were a number of hickups and issues along the way. I<br>
think that Mark (oslo PTL) can elaborate a little more on this. Timing is<br>
essential.<br>
</blockquote>
<br></div>
+1<br>
<br>
I'd also add that pulling oslo db into its own package means that<br>
projects using Oslo's db code have to be migrated as well. I think<br>
it's a bit late for that. The focus should be on making sure current<br>
code is stable enough for the not-so-far release.<br>
<br>
Thanks for the hard work!<br>
FF<span><font color="#888888"><br>
<br>
-- <br>
@flaper87<br>
Flavio Percoco</font></span><div><div><br>
<br>
______________________________<u></u>_________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org" target="_blank">OpenStack-dev@lists.openstack.<u></u>org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/<u></u>cgi-bin/mailman/listinfo/<u></u>openstack-dev</a><br>
</div></div></blockquote></div><br></div></div></div>