openstack/keystone
Code Issues Proposed changes

Remove the deprecated compute_port option

Browse Source 
The compute_port option has been marked deprecated and should be remove in
Liberty, and it is useless now.

Change-Id: I5832ec450e3fd89c9159c5ff333af7c100a5810c
Partially implements: blueprint removed-as-of-liberty
This commit is contained in:
liu-sheng 2015-05-15 14:31:26 +08:00
parent c5a2d9482b
commit b5baf63e1a
2 changed files with 1 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
keystone/catalog/core.py
View File

@ -50,7 +50,7 @@ def format_url(url, substitutions, silent_keyerror_failures=None):
WHITELISTED_PROPERTIES = [
'tenant_id', 'user_id', 'public_bind_host', 'admin_bind_host',
'compute_host', 'compute_port', 'admin_port', 'public_port',
'compute_host', 'admin_port', 'public_port',
'public_endpoint', 'admin_endpoint', ]
substitutions = utils.WhiteListedItemFilter(

8
keystone/common/config.py
View File

@ -32,14 +32,6 @@ FILE_OPTIONS = {
'AdminTokenAuthMiddleware from your paste '
'application pipelines (for example, in '
'keystone-paste.ini).'),
cfg.IntOpt('compute_port', default=8774,
help='(Deprecated) The port which the OpenStack Compute '
'service listens on. This option was only used for '
'string replacement in the templated catalog backend. '
'Templated catalogs should replace the '
'"$(compute_port)s" substitution with the static port '
'of the compute service. As of Juno, this option is '
'deprecated and will be removed in the L release.'),
cfg.StrOpt('public_endpoint',
help='The base public endpoint URL for Keystone that is '
'advertised to clients (NOTE: this does NOT affect '