diff options
| author | Roman Podolyaka <rpodolyaka@mirantis.com> | 2013-05-18 23:06:55 +0300 |
|---|---|---|
| committer | Roman Podolyaka <rpodolyaka@mirantis.com> | 2013-05-18 23:06:55 +0300 |
| commit | 19aaaf5ee166ae0276a4bc00d8bfba1fb3c7fc57 (patch) | |
| tree | af0d1cdeea28da0a882db93525f79a66b3df76bb /nova/db | |
| parent | f19571050a2ca3bf9dfa5d036ede23aca32bb604 (diff) | |
Fix require_context() decorators.
require_context() and require_admin_context() decorators
don't copy attributes of a decorated function (func_name,
__doc__, __module__) to a wrapper function. This makes
introspection and debugging harder and should be fixed.
Fixes bug 1181606.
Change-Id: I0926df92a40d3fc09b94a0c4b1ded63039abf18b
Diffstat (limited to 'nova/db')
| -rw-r--r-- | nova/db/sqlalchemy/api.py | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/nova/db/sqlalchemy/api.py b/nova/db/sqlalchemy/api.py index a35e8154d..53131331d 100644 --- a/nova/db/sqlalchemy/api.py +++ b/nova/db/sqlalchemy/api.py @@ -95,6 +95,7 @@ def require_admin_context(f): """ + @functools.wraps(f) def wrapper(*args, **kwargs): nova.context.require_admin_context(args[0]) return f(*args, **kwargs) @@ -112,6 +113,7 @@ def require_context(f): """ + @functools.wraps(f) def wrapper(*args, **kwargs): nova.context.require_context(args[0]) return f(*args, **kwargs) |