Use common code from Oslo for work with database

Registered by Viktor Serhieiev

There is common code for work with database in Oslo. It contains work around session, SQLAchemy engines, wrappers, etc.
We can use this code instead of Keystone implementation.

So we should:
1) Sync common code for work with database from Oslo
2) Use Oslo code instead of current project implementation.
   - work with sqlalchemy session and engines
   - use sqlalchemy migration utils, to make work with migrations more clear

This will close:
- https://bugs.launchpad.net/keystone/+bug/1272086

Blueprint information

Status:
Complete
Approver:
None
Priority:
Medium
Drafter:
Viktor Serhieiev
Direction:
Needs approval
Assignee:
Ilya Pekelny
Definition:
New
Series goal:
None
Implementation:
Implemented
Milestone target:
None
Started by
Viktor Serhieiev
Completed by
Morgan Fainberg

Related branches

Sprints

Whiteboard

(morganfainberg): We're completely over to using oslo.db at this point in Keystone. I'm going to mark this as implemented. (Oct 20, 2014)

Gerrit topic: https://review.openstack.org/#q,topic:bp/use-common-oslo-db-code,n,z

Addressed by: https://review.openstack.org/38029 (merged)
    Add common code from Oslo for work with database

Addressed by: https://review.openstack.org/38030 (abandoned)
    Use common Oslo code for work with database

Addressed by: https://review.openstack.org/44301 (merged)
    Use common db model class from Oslo

Addressed by: https://review.openstack.org/50767 (merged)
    Allow delete user or group at same time as role

Addressed by: https://review.openstack.org/49460 (merged)
    Use oslo.db sessions

Addressed by: https://review.openstack.org/51481 (merged)
    Enhance tests for delete_grant no user/group

Addressed by: https://review.openstack.org/61073
    Synchronize Keystone with oslo migration.py

Addressed by: https://review.openstack.org/59199 (abandoned)
    Sync openstack.common.test.py with Oslo

Addressed by: https://review.openstack.org/61609 (merged)
    Sync db.sqlalchemy from oslo-incubator

Gerrit topic: https://review.openstack.org/#q,topic:refactor_remove_base,n,z

Addressed by: https://review.openstack.org/63546
    Remove sql.Base.get_session

Addressed by: https://review.openstack.org/63547
    Move sql.Base.transaction

Addressed by: https://review.openstack.org/63548
    Remove sql.Base

Addressed by: https://review.openstack.org/58786 (merged)
    Synchronized with oslo db and db.sqlalchemy

Addressed by: https://review.openstack.org/65713
    Merge db.sqlalchemy from oslo-incubator af5f710

Addressed by: https://review.openstack.org/69136
    Don't mask the filter built-in

Addressed by: https://review.openstack.org/69137
    Move filter_query out of sql.Base

Addressed by: https://review.openstack.org/69142
    Cleanup sql.Base.filter_query

Addressed by: https://review.openstack.org/69143
    Remove unused and misspelled argument

Addressed by: https://review.openstack.org/69869
    `find_migrate_repo` refactoring

Addressed by: https://review.openstack.org/69593
    Correct `find_migrate_repo` usage

Addressed by: https://review.openstack.org/70408
    Synchronization with oslo migration.py (constant)

Addressed by: https://review.openstack.org/72088
    Correct error class in find_migrate_repo

Addressed by: https://review.openstack.org/72323
    Extension absence handling

Addressed by: https://review.openstack.org/74987
    Remove common.sql.migration

Addressed by: https://review.openstack.org/75928
    WIP: Sync with solo.db

(?)

Work Items

Work items:
Sync common code for work with database from Oslo: DONE
Work with sqlalchemy session and engines: DONE
Use sqlalchemy migration utils, to make work with migrations more clear: TODO

Dependency tree

* Blueprints in grey have been implemented.

This blueprint contains Public information 
Everyone can see this information.

Subscribers

No subscribers.