Add tpool proxy wrapper for DB-API calls
Current blueprint tracks the need for the new module, with notes to update nova and cinder
- it should just be about restoring the one missing option - CONF.database.
This was removed from oslo.db in patch [1] and we have no intentions to restore it there. oslo.db should not provide tpool implementation for the very same reason sqlalchemy does not provide one. From the point of view of oslo.db API we neither care whether you use OS threads, green threads, callbacks, processes or anything else for doing concurrency, nor we can make any assumptions about that. Also we don't want to force people to have eventlet as a runtime or even an install time dependency.
On another hand, it's a config option, and some people might be using it, so we can’t just remove it from OpenStack projects without breaking backwards compatibility.
Nova currently uses tpool.Proxy (eventlet implementation of the wrapper we used to provide in oslo.db) by the means of a helper class. It makes sense to put this code into incubator, so that it could be reused by other projects.
[1] https:/
Blueprint information
- Status:
- Complete
- Approver:
- Doug Hellmann
- Priority:
- High
- Drafter:
- Viktor Serhieiev
- Direction:
- Approved
- Assignee:
- Andrey Kurilin
- Definition:
- Approved
- Series goal:
- Accepted for juno
- Implementation:
-
Implemented
- Milestone target:
-
juno-2
- Started by
- Andrey Kurilin
- Completed by
- Andrey Kurilin
Related branches
Related bugs
Sprints
Whiteboard
Gerrit topic: https:/
Addressed by: https:/
Add eventlet.
Addressed by: https:/
Add add-tpool-