openstack/deb-glance
Code Issues Proposed changes

register the config generator default hook with the right name

Browse Source 
No config generator hooks should ever be registered with a name that
belongs to another project. In this case, using oslo.middleware.cors
means that *every other project* that loads the middleware gets this
application's defaults when the generator is run on a system with
everything installed (such as a dev box with devstack). Use the name
of the app instead, to ensure that the defaults are only set when this
app's sample config and documentation are being generated.

Change-Id: I6a8c7d44b9db9325003ff2fdb667b0ced7739e96
Signed-off-by: Doug Hellmann <doug@doughellmann.com>
This commit is contained in:
Doug Hellmann 2016-03-11 15:27:51 -05:00
parent ef127ffc8d
commit dde2c72d75
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
setup.cfg
View File

@ -46,7 +46,7 @@ oslo.config.opts =
glance.manage = glance.opts:list_manage_opts
glance.glare = glance.opts:list_artifacts_opts
oslo.config.opts.defaults =
oslo.middleware.cors = glance.common.config:set_cors_middleware_defaults
glance.api = glance.common.config:set_cors_middleware_defaults
glance.database.migration_backend =
sqlalchemy = oslo_db.sqlalchemy.migration
glance.database.metadata_backend =
@ -88,4 +88,4 @@ output_file = glance/locale/glance.pot
[pbr]
warnerrors = True
autodoc_tree_index_modules = True
autodoc_tree_index_modules = True