openstack/ironic
Code Issues Proposed changes

Merge "Don't hold on to the API request dbapi"

Browse Source
This commit is contained in:
Zuul 2023-08-18 02:23:16 +00:00 committed by Gerrit Code Review
commit e4a5691d33
1 changed files with 10 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
ironic/api/hooks.py
View File

@ -34,6 +34,10 @@ GLOBAL_REQ_ID = 'openstack.global_request_id'
ID_FORMAT = (r'^req-[a-f0-9]{8}-[a-f0-9]{4}-' ID_FORMAT = (r'^req-[a-f0-9]{8}-[a-f0-9]{4}-'
r'[a-f0-9]{4}-[a-f0-9]{4}-[a-f0-9]{12}$') r'[a-f0-9]{4}-[a-f0-9]{4}-[a-f0-9]{12}$')
# Call once, don't call on each request because it is
# a ton of extra overhead.
DBAPI = dbapi.get_instance()
def policy_deprecation_check(): def policy_deprecation_check():
global CHECKED_DEPRECATED_POLICY_ARGS global CHECKED_DEPRECATED_POLICY_ARGS
@ -77,7 +81,12 @@ class DBHook(hooks.PecanHook):
"""Attach the dbapi object to the request so controllers can get to it.""" """Attach the dbapi object to the request so controllers can get to it."""
def before(self, state): def before(self, state):
state.request.dbapi = dbapi.get_instance() state.request.dbapi = DBAPI
def after(self, state):
# Explicitly set to None since we don't need the DB connection
# after we're done processing the request.
state.request.dbapi = None
class ContextHook(hooks.PecanHook): class ContextHook(hooks.PecanHook):