Clean up use of ORM to remove the need for scoped_session.

3 files changed, 10 insertions, 9 deletions

diff --git a/nova/db/api.py b/nova/db/api.py
index 1d10b1987..fa937dab2 100644
--- a/nova/db/api.py
+++ b/nova/db/api.py
@@ -463,12 +463,12 @@ def security_group_get(context, security_group_id):
 
 def security_group_get_by_name(context, project_id, group_name):
     """Returns a security group with the specified name from a project"""
-    return IMPL.securitygroup_get_by_name(context, project_id, group_name)
+    return IMPL.security_group_get_by_name(context, project_id, group_name)
 
 
 def security_group_get_by_project(context, project_id):
     """Get all security groups belonging to a project"""
-    return IMPL.securitygroup_get_by_project(context, project_id)
+    return IMPL.security_group_get_by_project(context, project_id)
 
 
 def security_group_get_by_instance(context, instance_id):
@@ -478,7 +478,7 @@ def security_group_get_by_instance(context, instance_id):
 
 def securitygroup_exists(context, project_id, group_name):
     """Indicates if a group name exists in a project"""
-    return IMPL.securitygroup_exists(context, project_id, group_name)
+    return IMPL.security_group_exists(context, project_id, group_name)
 
 
 def security_group_create(context, values):
diff --git a/nova/db/sqlalchemy/api.py b/nova/db/sqlalchemy/api.py
index 11779e30c..038bb7f23 100644
--- a/nova/db/sqlalchemy/api.py
+++ b/nova/db/sqlalchemy/api.py
@@ -646,10 +646,11 @@ def security_group_get(_context, security_group_id):
                      ).get(security_group_id)
 
 
-def securitygroup_get_by_name(context, project_id, group_name):
+def security_group_get_by_name(context, project_id, group_name):
     session = get_session()
     group_ref = session.query(models.SecurityGroup
                       ).options(eagerload('rules')
+                      ).options(eagerload('instances')
                       ).filter_by(project_id=project_id
                       ).filter_by(name=group_name
                       ).filter_by(deleted=False
@@ -662,7 +663,7 @@ def securitygroup_get_by_name(context, project_id, group_name):
     return group_ref
     
 
-def securitygroup_get_by_project(_context, project_id):
+def security_group_get_by_project(_context, project_id):
     session = get_session()
     return session.query(models.SecurityGroup
                  ).options(eagerload('rules')
@@ -681,9 +682,9 @@ def security_group_get_by_instance(_context, instance_id):
                       ).all()
 
 
-def securitygroup_exists(_context, project_id, group_name):
+def security_group_exists(_context, project_id, group_name):
     try:
-        group = securitygroup_get_by_name(_context, project_id, group_name)
+        group = security_group_get_by_name(_context, project_id, group_name)
         return group != None
     except exception.NotFound:
         return False
diff --git a/nova/db/sqlalchemy/session.py b/nova/db/sqlalchemy/session.py
index fffbd3443..826754f6a 100644
--- a/nova/db/sqlalchemy/session.py
+++ b/nova/db/sqlalchemy/session.py
@@ -20,7 +20,7 @@ Session Handling for SQLAlchemy backend
 """
 
 from sqlalchemy import create_engine
-from sqlalchemy.orm import sessionmaker, scoped_session
+from sqlalchemy.orm import sessionmaker
 
 from nova import flags
 
@@ -36,7 +36,7 @@ def get_session(autocommit=True, expire_on_commit=False):
     if not _MAKER:
         if not _ENGINE:
             _ENGINE = create_engine(FLAGS.sql_connection, echo=False)
-        _MAKER = scoped_session(sessionmaker(bind=_ENGINE,
+        _MAKER = (sessionmaker(bind=_ENGINE,
                                 autocommit=autocommit,
                                 expire_on_commit=expire_on_commit))
     session =  _MAKER()