diff --git a/zaza/openstack/charm_tests/nova/tests.py b/zaza/openstack/charm_tests/nova/tests.py index 0f27e9f..03faaae 100644 --- a/zaza/openstack/charm_tests/nova/tests.py +++ b/zaza/openstack/charm_tests/nova/tests.py @@ -16,13 +16,15 @@ """Encapsulate nova testing.""" +import json import logging import unittest import zaza.model -import zaza.openstack.charm_tests.test_utils as test_utils import zaza.openstack.charm_tests.glance.setup as glance_setup +import zaza.openstack.charm_tests.test_utils as test_utils import zaza.openstack.configure.guest +import zaza.openstack.utilities.openstack as openstack_utils class BaseGuestCreateTest(unittest.TestCase): @@ -156,6 +158,266 @@ class NovaCompute(test_utils.OpenStackBaseTest): self.assertFalse(int(run['Code']) == 0) +class NovaCloudController(test_utils.OpenStackBaseTest): + """Run nova-cloud-controller specific tests.""" + + XENIAL_MITAKA = openstack_utils.get_os_release('xenial_mitaka') + XENIAL_OCATA = openstack_utils.get_os_release('xenial_ocata') + XENIAL_QUEENS = openstack_utils.get_os_release('xenial_queens') + BIONIC_QUEENS = openstack_utils.get_os_release('bionic_queens') + BIONIC_ROCKY = openstack_utils.get_os_release('bionic_rocky') + + @classmethod + def setUpClass(cls): + """Run class setup for running nova-cloud-controller tests.""" + super(NovaCloudController, cls).setUpClass() + cls.current_release = openstack_utils.get_os_release() + + @property + def services(self): + """Return a list of services for the selected OpenStack release.""" + services = ['nova-scheduler', 'nova-conductor'] + if self.current_release <= self.BIONIC_QUEENS: + services.append('nova-api-os-compute') + if self.current_release <= self.XENIAL_MITAKA: + services.append('nova-cert') + if self.current_release >= self.XENIAL_OCATA: + services.append('apache2') + return services + + def test_104_compute_api_functionality(self): + """Verify basic compute API functionality.""" + logging.info('Instantiating nova client...') + keystone_session = openstack_utils.get_overcloud_keystone_session() + nova = openstack_utils.get_nova_session_client(keystone_session) + + logging.info('Checking api functionality...') + + actual_service_names = [service.to_dict()['binary'] for service in + nova.services.list()] + for expected_service_name in ('nova-scheduler', 'nova-conductor', + 'nova-compute'): + assert(expected_service_name in actual_service_names) + + # Thanks to setup.create_flavors we should have a few flavors already: + assert(len(nova.flavors.list()) > 0) + + # Just checking it's not raising and returning an iterable: + assert(len(nova.servers.list()) >= 0) + + def test_106_compute_catalog_endpoints(self): + """Verify that the compute endpoints are present in the catalog.""" + overcloud_auth = openstack_utils.get_overcloud_auth() + keystone_client = openstack_utils.get_keystone_client( + overcloud_auth) + actual_endpoints = keystone_client.service_catalog.get_endpoints() + + logging.info('Checking compute endpoints...') + + if self.current_release < self.XENIAL_QUEENS: + actual_compute_endpoints = actual_endpoints['compute'][0] + for expected_url in ('internalURL', 'adminURL', 'publicURL'): + assert(expected_url in actual_compute_endpoints) + else: + actual_compute_interfaces = [endpoint['interface'] for endpoint in + actual_endpoints['compute']] + for expected_interface in ('internal', 'admin', 'public'): + assert(expected_interface in actual_compute_interfaces) + + def test_220_nova_metadata_propagate(self): + """Verify that the vendor-data settings are propagated. + + Change vendor-data-url and assert that change propagates to the correct + file and that services are restarted as a result + """ + if self.current_release < self.BIONIC_ROCKY: + logging.info("Feature didn't exist before Rocky. Nothing to test") + return + + # Expected default and alternate values + current_value = zaza.model.get_application_config( + 'nova-cloud-controller')['vendor-data-url']['value'] + new_value = 'http://some-other.url/vdata' + + set_default = {'vendor-data-url': current_value} + set_alternate = {'vendor-data-url': new_value} + default_entry = {'api': { + 'vendordata_dynamic_targets': [current_value]}} + alternate_entry = {'api': {'vendordata_dynamic_targets': [new_value]}} + + # Config file affected by juju set config change + conf_file = '/etc/nova/nova.conf' + + # Make config change, check for service restarts + logging.info( + 'Setting config on nova-cloud-controller to {}'.format( + set_alternate)) + self.restart_on_changed( + conf_file, + set_default, + set_alternate, + default_entry, + alternate_entry, + self.services) + + def test_302_api_rate_limiting_is_enabled(self): + """Check that API rate limiting is enabled.""" + logging.info('Checking api-paste config file data...') + zaza.model.block_until_oslo_config_entries_match( + 'nova-cloud-controller', '/etc/nova/api-paste.ini', { + 'filter:legacy_ratelimit': { + 'limits': ["( POST, '*', .*, 9999, MINUTE );"]}}) + + def test_310_pci_alias_config(self): + """Verify that the pci alias data is rendered properly. + + Change pci-alias and assert that change propagates to the correct + file and that services are restarted as a result + """ + logging.info('Checking pci aliases in nova config...') + + # Expected default and alternate values + current_value = zaza.model.get_application_config( + 'nova-cloud-controller')['pci-alias'] + try: + current_value = current_value['value'] + except KeyError: + current_value = None + new_value = '[{}, {}]'.format( + json.dumps({ + 'name': 'IntelNIC', + 'capability_type': 'pci', + 'product_id': '1111', + 'vendor_id': '8086', + 'device_type': 'type-PF' + }, sort_keys=True), + json.dumps({ + 'name': ' Cirrus Logic ', + 'capability_type': 'pci', + 'product_id': '0ff2', + 'vendor_id': '10de', + 'device_type': 'type-PCI' + }, sort_keys=True)) + + set_default = {'pci-alias': current_value} + set_alternate = {'pci-alias': new_value} + + expected_conf_section = 'DEFAULT' + expected_conf_key = 'pci_alias' + if self.current_release >= self.XENIAL_OCATA: + expected_conf_section = 'pci' + expected_conf_key = 'alias' + + default_entry = {expected_conf_section: {}} + alternate_entry = {expected_conf_section: { + expected_conf_key: [ + ('{"capability_type": "pci", "device_type": "type-PF", ' + '"name": "IntelNIC", "product_id": "1111", ' + '"vendor_id": "8086"}'), + ('{"capability_type": "pci", "device_type": "type-PCI", ' + '"name": " Cirrus Logic ", "product_id": "0ff2", ' + '"vendor_id": "10de"}')]}} + + # Config file affected by juju set config change + conf_file = '/etc/nova/nova.conf' + + # Make config change, check for service restarts + logging.info( + 'Setting config on nova-cloud-controller to {}'.format( + set_alternate)) + self.restart_on_changed( + conf_file, + set_default, + set_alternate, + default_entry, + alternate_entry, + self.services) + + def test_900_restart_on_config_change(self): + """Checking restart happens on config change. + + Change debug mode and assert that change propagates to the correct + file and that services are restarted as a result + """ + # Expected default and alternate values + current_value = zaza.model.get_application_config( + 'nova-cloud-controller')['debug']['value'] + new_value = str(not bool(current_value)).title() + current_value = str(current_value).title() + + set_default = {'debug': current_value} + set_alternate = {'debug': new_value} + default_entry = {'DEFAULT': {'debug': [current_value]}} + alternate_entry = {'DEFAULT': {'debug': [new_value]}} + + # Config file affected by juju set config change + conf_file = '/etc/nova/nova.conf' + + # Make config change, check for service restarts + logging.info( + 'Setting verbose on nova-cloud-controller {}'.format( + set_alternate)) + self.restart_on_changed( + conf_file, + set_default, + set_alternate, + default_entry, + alternate_entry, + self.services) + + def test_901_pause_resume(self): + """Run pause and resume tests. + + Pause service and check services are stopped then resume and check + they are started + """ + with self.pause_resume(self.services): + logging.info("Testing pause resume") + + def test_902_quota_settings(self): + """Verify that the quota settings are propagated. + + Change quota-instances and assert that change propagates to the correct + file and that services are restarted as a result + """ + # Expected default and alternate values + current_value = zaza.model.get_application_config( + 'nova-cloud-controller')['quota-instances'] + try: + current_value = current_value['value'] + except KeyError: + current_value = 0 + new_value = '20' + + set_default = {'quota-instances': current_value} + set_alternate = {'quota-instances': new_value} + + expected_conf_section = 'DEFAULT' + expected_conf_key = 'quota_instances' + if self.current_release >= self.XENIAL_OCATA: + expected_conf_section = 'quota' + expected_conf_key = 'instances' + + default_entry = {expected_conf_section: {}} + alternate_entry = {expected_conf_section: { + expected_conf_key: [new_value]}} + + # Config file affected by juju set config change + conf_file = '/etc/nova/nova.conf' + + # Make config change, check for service restarts + logging.info( + 'Setting config on nova-cloud-controller to {}'.format( + set_alternate)) + self.restart_on_changed( + conf_file, + set_default, + set_alternate, + default_entry, + alternate_entry, + self.services) + + class SecurityTests(test_utils.OpenStackBaseTest): """nova-compute and nova-cloud-controller security tests.""" diff --git a/zaza/openstack/charm_tests/test_utils.py b/zaza/openstack/charm_tests/test_utils.py index 82fed63..137f096 100644 --- a/zaza/openstack/charm_tests/test_utils.py +++ b/zaza/openstack/charm_tests/test_utils.py @@ -160,7 +160,7 @@ class BaseCharmTest(unittest.TestCase): Workaround: libjuju refuses to accept data with types other than strings - through the zuzu.model.set_application_config + through the zaza.model.set_application_config :param config: Config dictionary with any typed values :type config: Dict[str,Any] @@ -317,7 +317,7 @@ class BaseCharmTest(unittest.TestCase): # If this is not an OSLO config file set default_config={} if default_entry: logging.debug( - 'Waiting for updates to propagate to '.format(config_file)) + 'Waiting for updates to propagate to {}'.format(config_file)) model.block_until_oslo_config_entries_match( self.application_name, config_file,