x/vmware-nsx
Code Issues Proposed changes

Change default eswitchd port to avoid conflict

Browse Source 
Previous default for eswitch/daemon_endpoint would set a TCP port (5001)
which is in the range of the well known ports.
Change to unreserved port (60001) will avoid that conflict.

Change-Id: I711c4659f497e44a6041ec9958835c1f2f4174a7
Closes-Bug: #1245885
This commit is contained in:
Irena Berezovsky
2013-12-05 13:03:57 +02:00
parent c85a888098
commit 0c6213ab69
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
neutron/plugins/mlnx/common/config.py
View File

@@ -43,7 +43,7 @@ eswitch_opts = [
help=_("Type of VM network interface: mlnx_direct or " help=_("Type of VM network interface: mlnx_direct or "
"hostdev")), "hostdev")),
cfg.StrOpt('daemon_endpoint', cfg.StrOpt('daemon_endpoint',
default='tcp://127.0.0.1:5001', default='tcp://127.0.0.1:60001',
help=_('eswitch daemon end point')), help=_('eswitch daemon end point')),
cfg.IntOpt('request_timeout', default=3000, cfg.IntOpt('request_timeout', default=3000,
help=_("The number of milliseconds the agent will wait for " help=_("The number of milliseconds the agent will wait for "

4
neutron/tests/unit/mlnx/test_defaults.py
View File

@@ -25,8 +25,6 @@ class ConfigurationTest(base.BaseTestCase):
def test_defaults(self): def test_defaults(self):
self.assertEqual(2, self.assertEqual(2,
cfg.CONF.AGENT.polling_interval) cfg.CONF.AGENT.polling_interval)
self.assertEqual('sudo',
cfg.CONF.AGENT.root_helper)
self.assertEqual('vlan', self.assertEqual('vlan',
cfg.CONF.MLNX.tenant_network_type) cfg.CONF.MLNX.tenant_network_type)
self.assertEqual(1, self.assertEqual(1,
@@ -34,3 +32,5 @@ class ConfigurationTest(base.BaseTestCase):
self.assertEqual(0, self.assertEqual(0,
len(cfg.CONF.ESWITCH. len(cfg.CONF.ESWITCH.
physical_interface_mappings)) physical_interface_mappings))
self.assertEqual('tcp://127.0.0.1:60001',
cfg.CONF.ESWITCH.daemon_endpoint)