x/vmware-nsx
Code Issues Proposed changes

Fix URL used in NCS mechanism driver sync_full() operation

Browse Source 
The URL was incorrect due to a simple argument-passing bug.

Change-Id: I2fabacdb2838022a1d187ceb70cb090c15457b7c
Closes-Bug: #1224981
This commit is contained in:
Luke Gorrie 2013-09-13 14:30:28 +00:00
parent 3f9fe8f6a5
commit 8eb142748d
2 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
neutron/plugins/ml2/drivers/mechanism_ncs.py
View File

@ -116,7 +116,7 @@ class NCSMechanismDriver(api.MechanismDriver):
json = {'openstack': {'network': networks,
'subnet': subnets,
'port': ports}}
self.sendjson('put', self.url, json)
self.sendjson('put', '', json)
self.out_of_sync = False
def sync_object(self, operation, object_type, context):

6
neutron/tests/unit/ml2/test_mechanism_ncs.py
View File

@ -14,6 +14,7 @@
# under the License.
from neutron.plugins.ml2 import config as config
from neutron.plugins.ml2.drivers import mechanism_ncs
from neutron.tests.unit import test_db_plugin as test_plugin
PLUGIN_NAME = 'neutron.plugins.ml2.plugin.Ml2Plugin'
@ -31,6 +32,11 @@ class NCSTestCase(test_plugin.NeutronDbPluginV2TestCase):
self.addCleanup(config.cfg.CONF.reset)
super(NCSTestCase, self).setUp(PLUGIN_NAME)
self.port_create_status = 'DOWN'
mechanism_ncs.NCSMechanismDriver.sendjson = self.check_sendjson
def check_sendjson(self, method, urlpath, obj):
# Confirm fix for bug #1224981
self.assertFalse(urlpath.startswith("http://"))
class NCSMechanismTestBasicGet(test_plugin.TestBasicGet, NCSTestCase):