yahoo-eng-team team mailing list archive

[Matt Riedemann <mriedem@xxxxxxxxxx>]

Public bug reported:

Most of the code in nova.db.sqlalchemy.utils is also in oslo-
incubator.openstack.common.db.sqlalchemy.utils, except for the
modify_indexes method which is not actually even used in the nova db
migration code anymore now that it's been compacted in icehouse.

Also, the oslo.db code has been getting synced over to nova more
frequently lately so rather than keep all of this duplicate code around
we should move nova to using the oslo utils code and drop the internal
nova one, with maybe moving the modify_indexes method to oslo first,
then sync back to nova and then drop nova.db.sqlalchemy.utils from nova.

We will have to make sure that there are no behavior differences in the
oslo code such that it would change the nova db schema, but we should be
able to use Dan Prince's nova/tools/db/schema_diff.py script to validate
that.

** Affects: nova
     Importance: Low
     Assignee: Matt Riedemann (mriedem)
         Status: Triaged


** Tags: db

** Changed in: nova
   Importance: Undecided => Low

** Changed in: nova
       Status: New => Triaged

** Changed in: nova
     Assignee: (unassigned) => Matt Riedemann (mriedem)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1280055

Title:
  Replace nova.db.sqlalchemy.utils with
  openstack.common.db.sqlalchemy.utils

Status in OpenStack Compute (Nova):
  Triaged

Bug description:
  Most of the code in nova.db.sqlalchemy.utils is also in oslo-
  incubator.openstack.common.db.sqlalchemy.utils, except for the
  modify_indexes method which is not actually even used in the nova db
  migration code anymore now that it's been compacted in icehouse.

  Also, the oslo.db code has been getting synced over to nova more
  frequently lately so rather than keep all of this duplicate code
  around we should move nova to using the oslo utils code and drop the
  internal nova one, with maybe moving the modify_indexes method to oslo
  first, then sync back to nova and then drop nova.db.sqlalchemy.utils
  from nova.

  We will have to make sure that there are no behavior differences in
  the oslo code such that it would change the nova db schema, but we
  should be able to use Dan Prince's nova/tools/db/schema_diff.py script
  to validate that.

To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/1280055/+subscriptions