diff options
| author | Jenkins <jenkins@review.openstack.org> | 2012-12-05 13:37:05 +0000 |
|---|---|---|
| committer | Gerrit Code Review <review@openstack.org> | 2012-12-05 13:37:05 +0000 |
| commit | 0f56a837f2d0e778d4a71dee099c939b4cc5e91e (patch) | |
| tree | ed89f39a513a73012ec78daf2f0386a3084414fd | |
| parent | 5770146195ed9fcbb381faafffdb423432f6929d (diff) | |
| parent | 5393254bcce9ec825631ced2e80bda827c63092e (diff) | |
| download | nova-0f56a837f2d0e778d4a71dee099c939b4cc5e91e.tar.gz nova-0f56a837f2d0e778d4a71dee099c939b4cc5e91e.tar.xz nova-0f56a837f2d0e778d4a71dee099c939b4cc5e91e.zip | |
Merge "improve sessions for key_pair_(create,destroy)"
| -rw-r--r-- | nova/db/sqlalchemy/api.py | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/nova/db/sqlalchemy/api.py b/nova/db/sqlalchemy/api.py index c1b6e66dd..f2d804985 100644 --- a/nova/db/sqlalchemy/api.py +++ b/nova/db/sqlalchemy/api.py @@ -1987,16 +1987,16 @@ def key_pair_create(context, values): @require_context def key_pair_destroy(context, user_id, name): authorize_user_context(context, user_id) - session = get_session() - with session.begin(): - key_pair_ref = key_pair_get(context, user_id, name, session=session) - key_pair_ref.delete(session=session) + model_query(context, models.KeyPair).\ + filter_by(user_id=user_id).\ + filter_by(name=name).\ + delete() @require_context -def key_pair_get(context, user_id, name, session=None): +def key_pair_get(context, user_id, name): authorize_user_context(context, user_id) - result = model_query(context, models.KeyPair, session=session).\ + result = model_query(context, models.KeyPair).\ filter_by(user_id=user_id).\ filter_by(name=name).\ first() |
