Most of the OpenStack project makes use of lower case for
configuration group names. But, 'neutron' was having group
names in either cases. Bug states it is fine to have group
name in upper case for [DEFAULT] but needs to be changed
for others. It was noted that [PLUMgridDirector] was having
characters.
Change-Id: I95f2dad6d5ae17f722202f4917b6ef460cd41e15
Closes-Bug: #1176683
In TestOpenvswitchSGServerRpcCallBack and NetworkBindingsTest
in OVS plugin tests, OVS plugin was not loaded properly.
* For NetworkBindingsTest, it can be fixed by changing setUp()
to take 'plugin' arguments in the base test classes
SecurityGroupDBTestCase and SGServerRpcCallBackMixinTestCase.
This fixes bug 1242510 in ML2 unit tests.
* For NetworkBindingsTest, it can be fixed by passing the plugin
class to super.setUp(). The test itself needs to be updated
because network binding is added when a network is created.
* NetworkBindingsTest in Linux Bridge plugin has the same issue,
so it is also fixed in this patch.
Closes-Bug: #1230083
Closes-Bug: #1242510
Change-Id: I914876225480585d822748c188e9b69d1adf46f3
Ensure that Conflict errors are not masked; also ensure
that coding errors do not mask NVP API errors.
Fixes bug #1244259
Change-Id: I27586d72af89fd39f6443dbab33f0e4e762b0c99
Avoid incorrect status change when deleting the pool.
We can check for the conditions prior putting the pool
to PENDING_DELETE state, in case delete conditions are met
it is safe to change the state to PENDING_DELETE.
Also, change create_vip and update_vip operations to respect
PENDING_DELETE and avoid race conditions.
Change-Id: I9f526901eb85bdb83cf4ff8131460eb592c900f8
Closes-Bug: #1242338
Previously when router ports were created the mac_address from neutron
was not passed to nvp so nvp generated it's own mac_address for the port.
This patch fixes this by passing the mac_address on the port to nvp.
Fixes bug: 1244481
Change-Id: I0d0e068923497a13915749f3781d54194cfcfe89
Removing workflows handling from the driver code.
Removing workflow related files
Modifying tests to support new behavior
Change-Id: Icbb6106db07e5b33c37192aa53d088e67bd4a795
Closes-bug: #1239288
This patch fixes an error that does not result in a test failure but
causes the following error to be logged in the test logs:
TypeError: <MagicMock name='LinuxBridgeManager().local_ip' id='666599248'>
is not JSON serializable
Related-Bug: #1234857
Change-Id: I65cdd997745d8b760031f8f32e3c10682ad2288e
Fix typo on major version. This was not caught during unit tests
so adds a few to ensure this does not regress.
Fix bug #1243862
Change-Id: Ie136aaa504939d8ac77dda07e3f955aed8c233f9
When NVP returns a 405 error code no exception is
raised, causing the plugin to assume that everything
went well when it didn't.
Fixes bug #1243411
Change-Id: If70db05b8c6950bfa88a36c4f0c20ee80419d31d
Using assertEqual/assertNotEqual to test if an element
is or is not None is too python2.4.
Our unit testing framework supports assertIsNone and assertIsNotNone
which were created for these types of tests.
Partial-bug: #1230028
Change-Id: Ie0bddae642f7973ee8b0e00164415f8d87de387e
The default for send_arp_for_ha was 3 and is now 0.
This patch changes the comment in l3_agent.ini to
show the new default.
Change-Id: I77370350ffee6ad7b94aa7db341c062ebc0b7d89
This patch removes the previous check that verified the mocked
looping call had an exact number of calls, and replaces it with a
simpler check that just verifies that the mock has been called.
As a consequence, this patch also shortens the test duration.
Closes-Bug: 1241969
Change-Id: Ia7bc66e1f2f631e46ddcbe0eacc1d5b2481274cb