summaryrefslogtreecommitdiffstats
path: root/nova/tests
diff options
context:
space:
mode:
authorVishvananda Ishaya <vishvananda@gmail.com>2012-03-08 12:53:44 -0800
committerVishvananda Ishaya <vishvananda@gmail.com>2012-03-12 00:14:50 -0700
commit33def9e714fbd13a6dc4b755ade4841c971f7ae5 (patch)
tree924e88e900a73855ffc8df5330c32f6ba9092462 /nova/tests
parent4a144270c82949189ff43028a55272e92dc2839d (diff)
Fix live-migration in multi_host network
* call teardown after live migration * call update a second time after migration for dhcp * moves the instance state update into post_live_migrate * completes the fix for bug 939060 * fixes bug 947326 Change-Id: I042567573b9bb46381c5447aa08e83cd1916b225
Diffstat (limited to 'nova/tests')
-rw-r--r--nova/tests/test_compute.py16
1 files changed, 7 insertions, 9 deletions
diff --git a/nova/tests/test_compute.py b/nova/tests/test_compute.py
index bcfe90e0e..f30fefeba 100644
--- a/nova/tests/test_compute.py
+++ b/nova/tests/test_compute.py
@@ -1387,26 +1387,24 @@ class ComputeTestCase(BaseTestCase):
self.mox.StubOutWithMock(self.compute.driver, 'unfilter_instance')
self.compute.driver.unfilter_instance(i_ref, [])
self.mox.StubOutWithMock(rpc, 'call')
- rpc.call(c, 'network', {'method': 'setup_networks_on_host',
- 'args': {'instance_id': instance_id,
- 'host': self.compute.host,
- 'teardown': True}})
rpc.call(c, db.queue_get_for(c, FLAGS.compute_topic, dest),
{"method": "post_live_migration_at_destination",
"args": {'instance_id': i_ref['id'], 'block_migration': False}})
self.mox.StubOutWithMock(self.compute.driver, 'unplug_vifs')
self.compute.driver.unplug_vifs(i_ref, [])
+ rpc.call(c, 'network', {'method': 'setup_networks_on_host',
+ 'args': {'instance_id': instance_id,
+ 'host': self.compute.host,
+ 'teardown': True}})
# start test
self.mox.ReplayAll()
self.compute.post_live_migration(c, i_ref, dest)
- # make sure every data is rewritten to destinatioin hostname.
- i_ref = db.instance_get(c, i_ref['id'])
- c1 = (i_ref['host'] == dest)
+ # make sure floating ips are rewritten to destinatioin hostname.
flo_refs = db.floating_ip_get_all_by_host(c, dest)
- c2 = (len(flo_refs) != 0 and flo_refs[0]['address'] == flo_addr)
- self.assertTrue(c1 and c2)
+ self.assertTrue(flo_refs)
+ self.assertEqual(flo_refs[0]['address'], flo_addr)
# cleanup
db.instance_destroy(c, instance_id)