Hello Matt. Thanks a lot for your working on this! In my opinion, these steps are correct. Please see a few minor notes below.
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 2,3 - Looks good to me :) 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). If you will have any issues or questions, please feel free to ping me or Roman Podoliaka via irc or e-mail. 5 - ??? 6 - Profit! [1] https://blueprints.launchpad.net/oslo/+spec/oslo-db-lib Thanks, Victor On Mon, May 5, 2014 at 5:47 PM, Matt Riedemann <[email protected]>wrote: > Just wanted to get some thoughts down while they are in my head this > morning. > > 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. > > 1. Looks like it's not in global-requirements yet [2], so that's probably > a first step. > > 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. > > 3. Rip this [4] out of oslo.db once migrate is updated and released. > > 4. Replace nova.openstack.common.db with oslo.db. > > 5. ??? > > 6. Profit! > > Did I miss anything? > > [1] http://git.openstack.org/cgit/openstack/oslo.db/ > [2] http://git.openstack.org/cgit/openstack/requirements/tree/ > global-requirements.txt > [3] https://review.openstack.org/#/c/87773/ > [4] https://review.openstack.org/#/c/31016/ > > -- > > Thanks, > > Matt Riedemann > > > _______________________________________________ > OpenStack-dev mailing list > [email protected] > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev >
_______________________________________________ OpenStack-dev mailing list [email protected] http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
