func tests: move _run_periodics() into base class

There are two almost identical implementations of the _run_periodics()
helper - and a third one would have joined them in a subsequent patch,
if not for this patch. This patch moves the _run_periodics() to the
base test class. In addition, _run_periodics() depends on the
self.computes dict used for compute service tracking. The method that
populates that dict, _start_compute(), is therefore also moved to the
base class.

This enables some light refactoring of existing tests that need
either the _run_periodics() helper, or the compute service tracking.

In addition, a needless override of _start_compute() in
test_aggregates that provided no added value is removed. This is done
to avoid any potential confusion around _start_compute()'s role.

Change-Id: I36dd64dc272ea1743995b3b696323a9431666489

safdasdf

Change-Id: I33d8ac0a1cae0b2d275a21287d5e44c008a68122
This commit is contained in:
Artom Lifshitz 2020-02-03 14:36:35 -05:00
parent 139d9b63b5
commit ee05cd8b9e
13 changed files with 62 additions and 104 deletions

View File

@ -76,6 +76,7 @@ logging.register_options(CONF)
CONF.set_override('use_stderr', False)
logging.setup(CONF, 'nova')
cache.configure(CONF)
LOG = logging.getLogger(__name__)
_TRUE_VALUES = ('True', 'true', '1', 'yes')
CELL1_NAME = 'cell1'
@ -233,6 +234,7 @@ class TestCase(base.BaseTestCase):
context.CELL_CACHE = {}
context.CELLS = []
self.computes = {}
self.cell_mappings = {}
self.host_mappings = {}
# NOTE(danms): If the test claims to want to set up the database
@ -376,7 +378,7 @@ class TestCase(base.BaseTestCase):
for k, v in kw.items():
CONF.set_override(k, v, group)
def start_service(self, name, host=None, **kwargs):
def start_service(self, name, host=None, cell_name=None, **kwargs):
# Disallow starting multiple scheduler services
if name == 'scheduler' and self._service_fixture_count[name]:
raise TestingException("Duplicate start_service(%s)!" % name)
@ -393,7 +395,7 @@ class TestCase(base.BaseTestCase):
# otherwise we'll fail to update the scheduler while running
# the compute node startup routines below.
ctxt = context.get_context()
cell_name = kwargs.pop('cell', CELL1_NAME) or CELL1_NAME
cell_name = cell_name or CELL1_NAME
cell = self.cell_mappings[cell_name]
if (host or name) not in self.host_mappings:
# NOTE(gibi): If the HostMapping does not exists then this is
@ -421,6 +423,36 @@ class TestCase(base.BaseTestCase):
return svc.service
def _start_compute(self, host, cell_name=None):
"""Start a nova compute service on the given host
:param host: the name of the host that will be associated to the
compute service.
:param cell_name: optional name of the cell in which to start the
compute service
:return: the nova compute service object
"""
compute = self.start_service('compute', host=host, cell_name=cell_name)
self.computes[host] = compute
return compute
def _run_periodics(self):
"""Run the update_available_resource task on every compute manager
This runs periodics on the computes in an undefined order; some child
class redefine this function to force a specific order.
"""
ctx = context.get_admin_context()
for host, compute in self.computes.items():
LOG.info('Running periodic for compute (%s)', host)
# Make sure the context is targeted to the proper cell database
# for multi-cell tests.
with context.target_cell(
ctx, self.host_mappings[host].cell_mapping) as cctxt:
compute.manager.update_available_resource(cctxt)
LOG.info('Finished with periodics')
def restart_compute_service(self, compute, keep_hypervisor_state=True):
"""Stops the service and starts a new one to have realistic restart
@ -464,10 +496,10 @@ class TestCase(base.BaseTestCase):
'nova.virt.driver.load_compute_driver') as load_driver:
load_driver.return_value = old_driver
new_compute = self.start_service(
'compute', host=compute.host, cell=cell_name)
'compute', host=compute.host, cell_name=cell_name)
else:
new_compute = self.start_service(
'compute', host=compute.host, cell=cell_name)
'compute', host=compute.host, cell_name=cell_name)
return new_compute

View File

@ -34,11 +34,11 @@ class ImageCacheTest(test.TestCase):
self.compute1 = self.start_service('compute', host='compute1')
self.compute2 = self.start_service('compute', host='compute2')
self.compute3 = self.start_service('compute', host='compute3',
cell='cell2')
cell_name='cell2')
self.compute4 = self.start_service('compute', host='compute4',
cell='cell2')
cell_name='cell2')
self.compute5 = self.start_service('compute', host='compute5',
cell='cell2')
cell_name='cell2')
cell2 = self.cell_mappings['cell2']
with context.target_cell(self.context, cell2) as cctxt:

View File

@ -70,9 +70,7 @@ class VirtualInterfaceListMigrationTestCase(
fake_network.set_stub_network_methods(self)
self.cells = objects.CellMappingList.get_all(self.context)
compute_cell0 = self.start_service(
'compute', host='compute2', cell='cell0')
self.computes = [compute_cell0, self.compute]
self._start_compute('compute2')
self.instances = []
def _create_instances(self, pre_newton=2, deleted=0, total=5,
@ -93,7 +91,7 @@ class VirtualInterfaceListMigrationTestCase(
flavor=flavor,
created_at=datetime.datetime(1985, 10, 25, 1, 21, 0),
launched_at=datetime.datetime(1985, 10, 25, 1, 22, 0),
host=self.computes[0].host,
host=self.computes['compute2'].host,
hostname='%s-inst%i' % (target_cell.name, i))
inst.create()

View File

@ -377,7 +377,7 @@ class _IntegratedTestBase(test.TestCase, InstanceHelperMixin):
self.addCleanup(nova.tests.unit.image.fake.FakeImageService_reset)
def _setup_compute_service(self):
return self.start_service('compute')
return self._start_compute('compute')
def _setup_scheduler_service(self):
return self.start_service('scheduler')
@ -504,21 +504,6 @@ class ProviderUsageBaseTestCase(test.TestCase, InstanceHelperMixin):
self.addCleanup(nova.tests.unit.image.fake.FakeImageService_reset)
self.computes = {}
def _start_compute(self, host, cell_name=None):
"""Start a nova compute service on the given host
:param host: the name of the host that will be associated to the
compute service.
:param cell_name: optional name of the cell in which to start the
compute service (defaults to cell1)
:return: the nova compute service object
"""
compute = self.start_service('compute', host=host, cell=cell_name)
self.computes[host] = compute
return compute
def _get_provider_uuid_by_host(self, host):
# NOTE(gibi): the compute node id is the same as the compute node
# provider uuid on that compute
@ -856,23 +841,6 @@ class ProviderUsageBaseTestCase(test.TestCase, InstanceHelperMixin):
self.assertEqual(0, len(allocations))
return migration_uuid
def _run_periodics(self):
"""Run the update_available_resource task on every compute manager
This runs periodics on the computes in an undefined order; some child
class redefined this function to force a specific order.
"""
ctx = context.get_admin_context()
for host, compute in self.computes.items():
LOG.info('Running periodic for compute (%s)', host)
# Make sure the context is targeted to the proper cell database
# for multi-cell tests.
with context.target_cell(
ctx, self.host_mappings[host].cell_mapping) as cctxt:
compute.manager.update_available_resource(cctxt)
LOG.info('Finished with periodics')
def _move_and_check_allocations(self, server, request, old_flavor,
new_flavor, source_rp_uuid, dest_rp_uuid):
self.api.post_server_action(server['id'], request)

View File

@ -60,11 +60,8 @@ class TestEvacuationWithSourceReturningDuringRebuild(
self.start_service('scheduler')
# Start two computes
self.computes = {}
self.computes['host1'] = self.start_service('compute', host='host1')
self.computes['host2'] = self.start_service('compute', host='host2')
self._start_compute('host1')
self._start_compute('host2')
self.image_id = self.api.get_images()[0]['id']
self.flavor_id = self.api.get_flavors()[0]['id']

View File

@ -49,9 +49,7 @@ class MultiCellEvacuateTestCase(integrated_helpers._IntegratedTestBase):
"""
host_to_cell = {'host1': 'cell1', 'host2': 'cell2', 'host3': 'cell1'}
for host, cell in host_to_cell.items():
svc = self.start_service('compute', host=host, cell=cell)
# Set an attribute so we can access this service later.
setattr(self, host, svc)
self._start_compute(host, cell_name=cell)
def test_evacuate_multi_cell(self):
# Create a server which should land on host1 since it has the highest
@ -62,7 +60,7 @@ class MultiCellEvacuateTestCase(integrated_helpers._IntegratedTestBase):
self.assertEqual('host1', server['OS-EXT-SRV-ATTR:host'])
# Disable the host on which the server is now running.
self.host1.stop()
self.computes['host1'].stop()
self.api.force_down_service('host1', 'nova-compute', forced_down=True)
# Now evacuate the server which should send it to host3 since it is

View File

@ -68,10 +68,8 @@ class MissingReqSpecInstanceGroupUUIDTestCase(
self.start_service('scheduler')
# Start two computes, one where the server will be created and another
# where we'll cold migrate it.
self.computes = {} # keep track of the compute services per host name
for host in ('host1', 'host2'):
compute_service = self.start_service('compute', host=host)
self.computes[host] = compute_service
self._start_compute('host1')
self._start_compute('host2')
def test_cold_migrate_reschedule(self):
# Create an anti-affinity group for the server.

View File

@ -178,17 +178,6 @@ class AggregateRequestFiltersTest(
# Aggregate with neither host
self._create_aggregate('no-hosts')
def _start_compute(self, host):
"""Start a nova compute service on the given host
:param host: the name of the host that will be associated to the
compute service.
:return: the nova compute service object
"""
compute = self.start_service('compute', host=host)
self.computes[host] = compute
return compute
def _create_aggregate(self, name):
agg = self.admin_api.post_aggregate({'aggregate': {'name': name}})
self.aggregates[name] = agg
@ -840,9 +829,6 @@ class TestAggregateFiltersTogether(AggregateRequestFiltersTest):
class TestAggregateMultiTenancyIsolationFilter(
test.TestCase, integrated_helpers.InstanceHelperMixin):
def _start_compute(self, host):
self.start_service('compute', host=host)
def setUp(self):
super(TestAggregateMultiTenancyIsolationFilter, self).setUp()
# Stub out glance, placement and neutron.
@ -864,7 +850,7 @@ class TestAggregateMultiTenancyIsolationFilter(
self.flags(enabled_filters=enabled_filters, group='filter_scheduler')
self.start_service('scheduler')
for host in ('host1', 'host2'):
self._start_compute(host)
self.start_service('compute', host=host)
def test_aggregate_multitenancy_isolation_filter(self):
"""Tests common scenarios with the AggregateMultiTenancyIsolation

View File

@ -1467,9 +1467,9 @@ class TestDBArchiveDeletedRowsMultiCell(integrated_helpers.InstanceHelperMixin,
# Start two compute services, one per cell
self.compute1 = self.start_service('compute', host='host1',
cell='cell1')
cell_name='cell1')
self.compute2 = self.start_service('compute', host='host2',
cell='cell2')
cell_name='cell2')
def test_archive_deleted_rows(self):
admin_context = context.get_admin_context(read_deleted='yes')

View File

@ -64,8 +64,8 @@ class MultiCellSchedulerTestCase(test.TestCase,
in different cells and make sure that migration fails with NoValidHost.
"""
# Hosts in different cells
self.start_service('compute', host='compute1', cell=CELL1_NAME)
self.start_service('compute', host='compute2', cell=CELL2_NAME)
self.start_service('compute', host='compute1', cell_name=CELL1_NAME)
self.start_service('compute', host='compute2', cell_name=CELL2_NAME)
_, server = self._test_create_and_migrate(expected_status=202)
# The instance action should have failed with details.
@ -79,11 +79,11 @@ class MultiCellSchedulerTestCase(test.TestCase,
migration is allowed.
"""
# Hosts in the same cell
self.start_service('compute', host='compute1', cell=CELL1_NAME)
self.start_service('compute', host='compute2', cell=CELL1_NAME)
self.start_service('compute', host='compute1', cell_name=CELL1_NAME)
self.start_service('compute', host='compute2', cell_name=CELL1_NAME)
# Create another host just so it looks like we have hosts in
# both cells
self.start_service('compute', host='compute3', cell=CELL2_NAME)
self.start_service('compute', host='compute3', cell_name=CELL2_NAME)
# Force the server onto compute1 in cell1 so we do not accidentally
# land on compute3 in cell2 and fail to migrate.

View File

@ -867,9 +867,9 @@ class ServerGroupTestMultiCell(ServerGroupTestBase):
super(ServerGroupTestMultiCell, self).setUp()
# Start two compute services, one per cell
self.compute1 = self.start_service('compute', host='host1',
cell='cell1')
cell_name='cell1')
self.compute2 = self.start_service('compute', host='host2',
cell='cell2')
cell_name='cell2')
# This is needed to find a server that is still booting with multiple
# cells, while waiting for the state change to ACTIVE. See the
# _get_instance method in the compute/api for details.

View File

@ -74,7 +74,6 @@ class ServersTestBase(integrated_helpers._IntegratedTestBase):
_min_count_parameter = 'min_count'
def setUp(self):
self.computes = {}
super(ServersTestBase, self).setUp()
def _get_access_ips_params(self):
@ -102,23 +101,6 @@ class ServersTest(ServersTestBase):
node.hypervisor_hostname: int(node.stats.get('failed_builds', 0))
for node in computes}
def _run_periodics(self):
"""Run the update_available_resource task on every compute manager
This runs periodics on the computes in an undefined order; some child
class redefined this function to force a specific order.
"""
if self.compute.host not in self.computes:
self.computes[self.compute.host] = self.compute
ctx = context.get_admin_context()
for compute in self.computes.values():
LOG.info('Running periodic for compute (%s)',
compute.manager.host)
compute.manager.update_available_resource(ctx)
LOG.info('Finished with periodics')
def test_create_server_with_error(self):
# Create a server which will enter error state.
@ -170,8 +152,7 @@ class ServersTest(ServersTestBase):
def _test_create_server_with_error_with_retries(self):
# Create a server which will enter error state.
self.compute2 = self.start_service('compute', host='host2')
self.computes['compute2'] = self.compute2
self._start_compute('host2')
fails = []
@ -4580,7 +4561,7 @@ class ServerTestV256MultiCellTestCase(ServerTestV256Common):
'host2': 'cell2'}
for host in sorted(host_to_cell_mappings):
self.start_service('compute', host=host,
cell=host_to_cell_mappings[host])
cell_name=host_to_cell_mappings[host])
def test_migrate_server_to_host_in_different_cell(self):
# We target host1 specifically so that we have a predictable target for

View File

@ -2051,8 +2051,8 @@ class ConductorTaskTestCase(_BaseTaskTestCase, test_compute.BaseTestCase):
params = self.params
# The cells are created in the base TestCase setup.
self.start_service('compute', host='host1', cell='cell1')
self.start_service('compute', host='host2', cell='cell2')
self.start_service('compute', host='host1', cell_name='cell1')
self.start_service('compute', host='host2', cell_name='cell2')
get_hostmapping.side_effect = self.host_mappings.values()