8ec8987c31
Currently, several plugins already check config options at __init__() for validity and will exit, if the settings are incorrect. However, most (all?) config option definitions have default values, so if the option is missing, a valid, but maybe unexpected value will be used. This is what occurred in the bug. The proposed fix is to take a config option, sql_connection, which is used by many plugins, and remove the default value. Then, at init time, when the config option is used in configure_db(), a check is made for the value. If the value is not set, a warning is logged and the value is set to the default, for db/api.py. It is expected that this will be the only module to consume this config option. Added UT to check that log warning is issued. Also, changed the timing so that the test takes 0.25 secs vs 12 secs. Removed UTs in two plugin tests that checked the default value for sql_connection. Other alternatives explored in previous patches, were to either raise an exception, or mark this config option as "required". This resulted in a large number of changes to tests, and required config overrides in plugins that imported quantum.db.api, but did not use sql_connection. In order to keep this solution (of this log-hanging fruit) fix, the proposed, simpler change is being made. Some cleanup to the Cisco plugin test case was also made, so that the mock was more in line with what production code does. bug 1059923 Change-Id: I8c2a4e05231ac4e172d0dccece067e6fdb354341
142 lines
5.1 KiB
Python
142 lines
5.1 KiB
Python
# Copyright (c) 2012 OpenStack, LLC.
|
|
#
|
|
# Licensed under the Apache License, Version 2.0 (the "License");
|
|
# you may not use this file except in compliance with the License.
|
|
# You may obtain a copy of the License at
|
|
#
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
#
|
|
# Unless required by applicable law or agreed to in writing, software
|
|
# distributed under the License is distributed on an "AS IS" BASIS,
|
|
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or
|
|
# implied.
|
|
# See the License for the specific language governing permissions and
|
|
# limitations under the License.
|
|
|
|
import mock
|
|
import unittest
|
|
|
|
from quantum.db import api as db
|
|
from quantum.openstack.common import importutils
|
|
from quantum.plugins.cisco.common import cisco_constants as const
|
|
from quantum.plugins.cisco.db import network_models_v2
|
|
from quantum.plugins.cisco.nexus import cisco_nexus_plugin_v2
|
|
|
|
|
|
NEXUS_IP_ADDRESS = '1.1.1.1'
|
|
NEXUS_USERNAME = 'username'
|
|
NEXUS_PASSWORD = 'password'
|
|
HOSTNAME = 'testhost'
|
|
INSTANCE = 'testvm'
|
|
NEXUS_PORTS = '1/10'
|
|
NEXUS_SSH_PORT = '22'
|
|
NEXUS_DRIVER = ('quantum.plugins.cisco.tests.unit.v2.nexus.'
|
|
'fake_nexus_driver.CiscoNEXUSFakeDriver')
|
|
|
|
|
|
class TestCiscoNexusPlugin(unittest.TestCase):
|
|
|
|
def setUp(self):
|
|
"""
|
|
Set up function
|
|
"""
|
|
self.tenant_id = "test_tenant_cisco1"
|
|
self.net_name = "test_network_cisco1"
|
|
self.net_id = 000007
|
|
self.vlan_name = "q-" + str(self.net_id) + "vlan"
|
|
self.vlan_id = 267
|
|
self.second_net_name = "test_network_cisco2"
|
|
self.second_net_id = 000005
|
|
self.second_vlan_name = "q-" + str(self.second_net_id) + "vlan"
|
|
self.second_vlan_id = 265
|
|
self._nexus_switches = {
|
|
NEXUS_IP_ADDRESS: {
|
|
HOSTNAME: {
|
|
'ports': NEXUS_PORTS,
|
|
},
|
|
'ssh_port': {
|
|
'ssh_port': NEXUS_SSH_PORT
|
|
}
|
|
}
|
|
}
|
|
self._hostname = HOSTNAME
|
|
|
|
def new_nexus_init(self):
|
|
self._client = importutils.import_object(NEXUS_DRIVER)
|
|
self._nexus_ip = NEXUS_IP_ADDRESS
|
|
self._nexus_username = NEXUS_USERNAME
|
|
self._nexus_password = NEXUS_PASSWORD
|
|
self._nexus_ports = NEXUS_PORTS
|
|
self._nexus_ssh_port = NEXUS_SSH_PORT
|
|
self.credentials = {
|
|
self._nexus_ip: {
|
|
'username': self._nexus_username,
|
|
'password': self._nexus_password
|
|
}
|
|
}
|
|
db.configure_db()
|
|
|
|
with mock.patch.object(cisco_nexus_plugin_v2.NexusPlugin,
|
|
'__init__', new=new_nexus_init):
|
|
self._cisco_nexus_plugin = cisco_nexus_plugin_v2.NexusPlugin()
|
|
self._cisco_nexus_plugin._nexus_switches = self._nexus_switches
|
|
|
|
def test_a_create_network(self):
|
|
"""
|
|
Tests creation of two new Virtual Network.
|
|
Tests deletion of one Virtual Network.
|
|
This would result the following -
|
|
The Nexus device should have only one network
|
|
vlan configured on it's plugin configured
|
|
interfaces.
|
|
If running this test individually, run
|
|
test_nexus_clear_vlan after this test to clean
|
|
up the second vlan created by this test.
|
|
"""
|
|
tenant_id = self.tenant_id
|
|
net_name = self.net_name
|
|
net_id = self.net_id
|
|
vlan_name = self.vlan_name
|
|
vlan_id = self.vlan_id
|
|
second_net_name = self.second_net_name
|
|
second_net_id = self.second_net_id
|
|
second_vlan_name = self.second_vlan_name
|
|
second_vlan_id = self.second_vlan_id
|
|
|
|
new_net_dict = self._cisco_nexus_plugin.create_network(
|
|
tenant_id, net_name, net_id,
|
|
vlan_name, vlan_id, self._hostname, INSTANCE)
|
|
self.assertEqual(new_net_dict[const.NET_ID], net_id)
|
|
self.assertEqual(new_net_dict[const.NET_NAME], self.net_name)
|
|
self.assertEqual(new_net_dict[const.NET_VLAN_NAME], self.vlan_name)
|
|
self.assertEqual(new_net_dict[const.NET_VLAN_ID], self.vlan_id)
|
|
|
|
new_net_dict = self._cisco_nexus_plugin.create_network(
|
|
tenant_id, second_net_name, second_net_id,
|
|
second_vlan_name, second_vlan_id, self._hostname,
|
|
INSTANCE)
|
|
|
|
self.assertEqual(new_net_dict[const.NET_ID], second_net_id)
|
|
self.assertEqual(new_net_dict[const.NET_NAME], self.second_net_name)
|
|
self.assertEqual(new_net_dict[const.NET_VLAN_NAME],
|
|
self.second_vlan_name)
|
|
self.assertEqual(new_net_dict[const.NET_VLAN_ID], self.second_vlan_id)
|
|
|
|
def test_b_nexus_delete_port(self):
|
|
"""
|
|
Test to clean up second vlan of nexus device
|
|
created by test_create_delete_network. This
|
|
test will fail if it is run individually.
|
|
"""
|
|
expected_instance_id = self._cisco_nexus_plugin.delete_port(
|
|
INSTANCE, self.second_vlan_id
|
|
)
|
|
|
|
self.assertEqual(expected_instance_id, INSTANCE)
|
|
|
|
def tearDown(self):
|
|
"""Clear the test environment"""
|
|
pass
|
|
# Remove database contents
|
|
#db.clear_db(network_models_v2.model_base.BASEV2)
|