x/gantt
Code Issues Proposed changes

Set node_availability_zone in XenAPIAggregateTestCase

Browse Source 
In XenAPIAggregateTestCase some service records
are directly injected to the db for testing with customized AZ
values . The AZ doesn't match the default availability
zone setting (nova), and may or may not be overwritten during
heart beat message update. It is mostly ignored but
sometimes make the test_remove_host_from_aggregate_error()
test fail, as discovered in the servicegroup test.
Set the AZ explicitly to get rid of the non-determinism.

Change-Id: I2ded5314480b2b7eb2254a58e3eb7ca9e3f8b32d
This commit is contained in:
Yun Mao
2012-11-18 15:54:46 -05:00
parent 121669dec2
commit f94346d58b
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
nova/tests/test_xenapi.py
View File

@@ -2076,7 +2076,8 @@ class XenAPIAggregateTestCase(stubs.XenAPITestBase):
firewall_driver='nova.virt.xenapi.firewall.' firewall_driver='nova.virt.xenapi.firewall.'
'Dom0IptablesFirewallDriver', 'Dom0IptablesFirewallDriver',
host='host', host='host',
compute_driver='xenapi.XenAPIDriver') compute_driver='xenapi.XenAPIDriver',
node_availability_zone='avail_zone1')
host_ref = xenapi_fake.get_all('host')[0] host_ref = xenapi_fake.get_all('host')[0]
stubs.stubout_session(self.stubs, stubs.FakeSessionForVMTests) stubs.stubout_session(self.stubs, stubs.FakeSessionForVMTests)
self.context = context.get_admin_context() self.context = context.get_admin_context()