diff options
author | Jenkins <jenkins@review.openstack.org> | 2013-06-24 20:29:36 +0000 |
---|---|---|
committer | Gerrit Code Review <review@openstack.org> | 2013-06-24 20:29:36 +0000 |
commit | 7d7568da5fd2b043e283697c44baf2afa1e63bea (patch) | |
tree | 02689507f43183cb6c4adf267a800fdada01bb23 | |
parent | d4c7e88c86eabdd48558717e20966d7b4d092eec (diff) | |
parent | 3d27a0e74bde3878af76414dc4020ac6f9f24e4e (diff) | |
download | nova-7d7568da5fd2b043e283697c44baf2afa1e63bea.tar.gz nova-7d7568da5fd2b043e283697c44baf2afa1e63bea.tar.xz nova-7d7568da5fd2b043e283697c44baf2afa1e63bea.zip |
Merge "Handle NoMoreFixedIps in _shutdown_instance."
-rwxr-xr-x | nova/compute/manager.py | 2 | ||||
-rw-r--r-- | nova/tests/compute/test_compute.py | 24 |
2 files changed, 25 insertions, 1 deletions
diff --git a/nova/compute/manager.py b/nova/compute/manager.py index e0b2f38e8..1a6479f7f 100755 --- a/nova/compute/manager.py +++ b/nova/compute/manager.py @@ -1345,7 +1345,7 @@ class ComputeManager(manager.SchedulerDependentManager): # get network info before tearing down try: network_info = self._get_instance_nw_info(context, instance) - except exception.NetworkNotFound: + except (exception.NetworkNotFound, exception.NoMoreFixedIps): network_info = network_model.NetworkInfo() # NOTE(vish) get bdms before destroying the instance diff --git a/nova/tests/compute/test_compute.py b/nova/tests/compute/test_compute.py index c887041f8..7953f8b63 100644 --- a/nova/tests/compute/test_compute.py +++ b/nova/tests/compute/test_compute.py @@ -1217,6 +1217,30 @@ class ComputeTestCase(BaseTestCase): LOG.info(_("After terminating instances: %s"), instances) self.assertEqual(len(instances), 0) + def test_terminate_no_fixed_ips(self): + # This is as reported in LP bug 1192893 + instance = jsonutils.to_primitive(self._create_fake_instance()) + + self.compute.run_instance(self.context, instance=instance) + + instances = db.instance_get_all(self.context) + LOG.info(_("Running instances: %s"), instances) + self.assertEqual(len(instances), 1) + + self.mox.StubOutWithMock(self.compute, '_get_instance_nw_info') + self.compute._get_instance_nw_info( + mox.IgnoreArg(), + mox.IgnoreArg()).AndRaise( + exception.NoMoreFixedIps() + ) + self.mox.ReplayAll() + + self.compute.terminate_instance(self.context, instance=instance) + + instances = db.instance_get_all(self.context) + LOG.info(_("After terminating instances: %s"), instances) + self.assertEqual(len(instances), 0) + def test_terminate_failure_leaves_task_state(self): """Ensure that a failure in terminate_instance does not result in the task state being reverted from DELETING (see LP 1046236). |