]> review.fuel-infra Code Review - openstack-build/heat-build.git/commit
Code Review / openstack-build / heat-build.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: e32feeb) | patch
Remove unnecesssary nova constructor calls in each manager API call path
authorSteven Dake <sdake@redhat.com>
Sun, 16 Sep 2012 19:40:02 +0000 (12:40 -0700)
committerSteven Dake <sdake@redhat.com>
Mon, 17 Sep 2012 19:13:04 +0000 (12:13 -0700)
commit28b4d3e37ccf1adffa0d3327736964d03dc6c472
treeebf234f2d143fa0e63fee4034451c62ae5880761tree | snapshot
parente32feeb448b388d31ac27e76e5bbad82b0e2d112commit | diff
Remove unnecesssary nova constructor calls in each manager API call path

auth.authenticate() creates a nova client object.  This nova client
object is never reused introducing a 40msec penalty for each API call.
The heat-api-cfn module authenticates incoming requests.  In a
properly secured environment (one where RPC is secured), the
auth.authenticate() calls serve no purpose.

Change-Id: I847af259f521a32a7bf3fda0ce3f72735f762a0c
Signed-off-by: Steven Dake <sdake@redhat.com>
heat/engine/auth.py diff | blob | history
heat/engine/manager.py diff | blob | history
heat/tests/test_engine_manager.py diff | blob | history