openstack
/
nova
Code Issues Proposed changes

[placement] fix typo in call to create auth middleware 

The typo meant that the intended non global config was not being
used, and instead the config was loaded in the normal way. This
worked because the config loaded by the project name is the same
as the config loaded otherwise.

There's no test coverage of this in unit and functional tests
because the NoAuthMiddleware is always used based on overridden
configuration. The gate tests that turn on placement will exercise
this code.

Change-Id: I9f1b8905fd9d5aadac61718e24c76b1b2f80410d
Closes-Bug: #1645306
This commit is contained in:
Chris Dent 2017-01-05 20:34:17 +00:00
parent 9c2308424e
commit aeecfe09c4
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
nova/api/openstack/placement/deploy.py
View File

@ -39,7 +39,7 @@ def deploy(conf, project_name):
else:
# Do not provide global conf to middleware here.
auth_middleware = auth_token.filter_factory(
{}, olso_config_project=project_name)
{}, oslo_config_project=project_name)
context_middleware = auth.PlacementKeystoneContext
req_id_middleware = request_id.RequestId