openstack
/
tacker
Code Issues Proposed changes

Change default port into 9890 

Previous 8888 is mostly used as alternative port for 8080, 80.
This patch changes tacker port into 9890 to avoid conflict.

Ranged 9890-9897 is unassinged according to
https://www.ietf.org/assignments/service-names-port-numbers/service-names-port-numbers.txt

Change-Id: Idbbb74bb15ec1310aff0393636a4952d6eb7b877
Closes-bug: 1588065
This commit is contained in:
gong yong sheng 2016-06-02 15:15:27 +08:00
parent 617f28a93e
commit 68b2fcb028
7 changed files with 10 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
devstack/lib/tacker
View File

@ -64,9 +64,9 @@ TACKER_CONF=$TACKER_CONF_DIR/tacker.conf
# Default name for Tacker database
TACKER_DB_NAME=${TACKER_DB_NAME:-tacker}
# Default Tacker Port
TACKER_PORT=${TACKER_PORT:-8888}
TACKER_PORT=${TACKER_PORT:-9890}
# Default Tacker Internal Port when using TLS proxy
TACKER_PORT_INT=${TACKER_PORT_INT:-18888} # TODO(FIX)
TACKER_PORT_INT=${TACKER_PORT_INT:-19890} # TODO(FIX)
# Default Tacker Host
TACKER_HOST=${TACKER_HOST:-$SERVICE_HOST}
# Default protocol

2
doc/source/devref/mano_api.rst
View File

@ -29,7 +29,7 @@ List API versions - Lists information about Tacker API version.
"id": "v1.0",
"links": [
{
"href": "http://10.18.160.13:8888/v1.0",
"href": "http://10.18.160.13:9890/v1.0",
"rel": "self"
}
]

6
doc/source/install/manual_installation.rst
View File

@ -104,9 +104,9 @@ d). Provide an endpoint to tacker service.
.. code-block:: console
openstack endpoint create --region RegionOne \
--publicurl 'http://<TACKER_NODE_IP>:8888/' \
--adminurl 'http://<TACKER_NODE_IP>:8888/' \
--internalurl 'http://<TACKER_NODE_IP>:8888/' <SERVICE-ID>
--publicurl 'http://<TACKER_NODE_IP>:9890/' \
--adminurl 'http://<TACKER_NODE_IP>:9890/' \
--internalurl 'http://<TACKER_NODE_IP>:9890/' <SERVICE-ID>
..

2
etc/tacker/tacker.conf
View File

@ -35,7 +35,7 @@ lock_path = $state_path/lock
# bind_host = 0.0.0.0
# Port the bind the API server to
# bind_port = 8888
# bind_port = 9890
# Path to the extensions. Note that this can be a colon-separated list of
# paths. For example:

2
tacker/common/config.py
View File

@ -34,7 +34,7 @@ LOG = logging.getLogger(__name__)
core_opts = [
cfg.StrOpt('bind_host', default='0.0.0.0',
help=_("The host IP to bind to")),
cfg.IntOpt('bind_port', default=8888,
cfg.IntOpt('bind_port', default=9890,
help=_("The port to bind to")),
cfg.StrOpt('api_paste_config', default="api-paste.ini",
help=_("The API paste config file to use")),

2
tacker/tests/etc/tacker.conf.test
View File

@ -9,7 +9,7 @@ debug = False
bind_host = 0.0.0.0
# Port the bind the API server to
bind_port = 8888
bind_port = 9890
# Path to the extensions
api_extensions_path = unit/extensions

2
tacker/tests/unit/test_config.py
View File

@ -26,7 +26,7 @@ class ConfigurationTest(base.BaseTestCase):
def test_defaults(self):
self.assertEqual('0.0.0.0', cfg.CONF.bind_host)
self.assertEqual(8888, cfg.CONF.bind_port)
self.assertEqual(9890, cfg.CONF.bind_port)
self.assertEqual('api-paste.ini', cfg.CONF.api_paste_config)
self.assertEqual('', cfg.CONF.api_extensions_path)
self.assertEqual('policy.json', cfg.CONF.policy_file)