<div dir="ltr"><div>Hello Matt.</div><div><br></div><div>Thanks a lot for your working on this! </div><div>In my opinion, these steps are correct. Please see a few minor notes below.</div><div><br></div><div>1 - Yes, you right, oslo.db is not in global-requirements now. Blueprint ``Split openstack.common.db code into a separate oslo.db library `` [1]  is not completed at the moment. Please see ``work items`` section in [1] for the actual bp state</div>
<div><br></div><div>2,3 - Looks good to me :)</div><div><br></div><div>4 - If you want, you test it locally even without oslo.db in global-requirements. You can replace nova.openstack.common.db with oslo.db on  and run unittests. This will help us to find and fix potential issues with porting to oslo.db (if any). </div>
<div>If you will have any issues or questions, please feel free to ping me or Roman Podoliaka via irc or e-mail.</div><div><br></div><div>5 - ???</div><div><br></div><div>6 - Profit! </div><div><br></div><div>[1] <a href="https://blueprints.launchpad.net/oslo/+spec/oslo-db-lib">https://blueprints.launchpad.net/oslo/+spec/oslo-db-lib</a></div>
<div><br></div><div>Thanks, </div><div>Victor </div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, May 5, 2014 at 5:47 PM, Matt Riedemann <span dir="ltr"><<a href="mailto:mriedem@linux.vnet.ibm.com" target="_blank">mriedem@linux.vnet.ibm.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Just wanted to get some thoughts down while they are in my head this morning.<br>
<br>
Oslo DB is now a library [1].  I'm trying to figure out what the steps are to getting Nova to using that so we can rip out the sync'ed common db code.<br>
<br>
1. Looks like it's not in global-requirements yet [2], so that's probably a first step.<br>
<br>
2. We'll want to cut a sqlalchemy-migrate release once this patch is merged [3]. This moves a decent chunk of unique constraint patch code out of oslo and into sqlalchemy-migrate where it belongs so we can run unit tests with sqlite to drop unique constraints.<br>

<br>
3. Rip this [4] out of oslo.db once migrate is updated and released.<br>
<br>
4. Replace nova.openstack.common.db with oslo.db.<br>
<br>
5. ???<br>
<br>
6. Profit!<br>
<br>
Did I miss anything?<br>
<br>
[1] <a href="http://git.openstack.org/cgit/openstack/oslo.db/" target="_blank">http://git.openstack.org/cgit/<u></u>openstack/oslo.db/</a><br>
[2] <a href="http://git.openstack.org/cgit/openstack/requirements/tree/global-requirements.txt" target="_blank">http://git.openstack.org/cgit/<u></u>openstack/requirements/tree/<u></u>global-requirements.txt</a><br>
[3] <a href="https://review.openstack.org/#/c/87773/" target="_blank">https://review.openstack.org/#<u></u>/c/87773/</a><br>
[4] <a href="https://review.openstack.org/#/c/31016/" target="_blank">https://review.openstack.org/#<u></u>/c/31016/</a><span class="HOEnZb"><font color="#888888"><br>
<br>
-- <br>
<br>
Thanks,<br>
<br>
Matt Riedemann<br>
<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>
</font></span></blockquote></div><br></div>