Share one engine for more than just sqlite in-memory

This fixes both bug 1162857 and bug 1162845.

Change-Id: Ica5155d3ac03779693d2d54d8e02c2a413c3d3ac

keystone/common/sql/core.py

@@ -30,7 +30,6 @@
 from keystone import config
 from keystone import exception
 from keystone.openstack.common import jsonutils
-from keystone import exception
 
 
 CONF = config.CONF
@@ -98,7 +97,6 @@ def set_global_engine(engine):
 
 
 def get_global_engine():
-    global GLOBAL_ENGINE
     return GLOBAL_ENGINE
 
 
@@ -250,6 +248,8 @@ def new_engine():
         # auto-build the db to support wsgi server w/ in-memory backend
         if allow_global_engine and CONF.sql.connection == 'sqlite://':
             ModelBase.metadata.create_all(bind=engine)
+
+        if allow_global_engine:
             set_global_engine(engine)
 
         return engine

keystone/common/sql/util.py

@@ -17,6 +17,7 @@
 import os
 import shutil
 
+from keystone.common.sql import core
 from keystone.common.sql import migration
 from keystone import config
 
@@ -38,5 +39,7 @@ def setup_test_database():
 
 
 def teardown_test_database():
+    core.set_global_engine(None)
+
     if os.path.exists('test.db.pristine'):
         os.unlink('test.db.pristine')