From 19aaaf5ee166ae0276a4bc00d8bfba1fb3c7fc57 Mon Sep 17 00:00:00 2001 From: Roman Podolyaka Date: Sat, 18 May 2013 23:06:55 +0300 Subject: 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 --- nova/db/sqlalchemy/api.py | 2 ++ 1 file changed, 2 insertions(+) (limited to 'nova/db') 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) -- cgit