diff options
| author | Dan Prince <dprince@redhat.com> | 2013-06-24 16:41:06 -0400 |
|---|---|---|
| committer | Dan Prince <dprince@redhat.com> | 2013-06-24 16:42:09 -0400 |
| commit | c9bb51c6da93a40f4057e99db146c109718bc3a6 (patch) | |
| tree | cfa9c8307f85af6f6367049b2d1e33edf72f3392 | |
| parent | 5fa2774e2715ad29dbf3c9342484cdc12560b2d0 (diff) | |
| download | nova-c9bb51c6da93a40f4057e99db146c109718bc3a6.tar.gz nova-c9bb51c6da93a40f4057e99db146c109718bc3a6.tar.xz nova-c9bb51c6da93a40f4057e99db146c109718bc3a6.zip | |
Revert XenApi virt driver should throw exception
This reverts commit 2cb31123302a1ec8c8dec0cb2ac288a2e3b19c68.
Fixes LP Bug #1194264.
Change-Id: Ib8f8a8cfdcc1b37bb87fee3983081154a4fa2b2b
| -rw-r--r-- | nova/tests/virt/xenapi/test_vmops.py | 11 | ||||
| -rw-r--r-- | nova/tests/virt/xenapi/test_xenapi.py | 5 | ||||
| -rw-r--r-- | nova/virt/xenapi/vmops.py | 6 |
3 files changed, 1 insertions, 21 deletions
diff --git a/nova/tests/virt/xenapi/test_vmops.py b/nova/tests/virt/xenapi/test_vmops.py index d2b5477eb..674d84882 100644 --- a/nova/tests/virt/xenapi/test_vmops.py +++ b/nova/tests/virt/xenapi/test_vmops.py @@ -171,17 +171,6 @@ class VMOpsTestCase(test.TestCase): self.assertEqual(self.make_plugin_call_count, 1) - def test_destroy_raises_when_shutdown_fails(self): - vm_ref = "vm_reference" - fake_instance = "instance" - - self.mox.StubOutWithMock(vm_utils, 'hard_shutdown_vm') - vm_utils.hard_shutdown_vm(self._session, fake_instance, - vm_ref).AndReturn(False) - self.mox.ReplayAll() - self.assertRaises(exception.InstancePowerOffFailure, - self._vmops._destroy, fake_instance, vm_ref) - class GetConsoleOutputTestCase(stubs.XenAPITestBase): def setUp(self): diff --git a/nova/tests/virt/xenapi/test_xenapi.py b/nova/tests/virt/xenapi/test_xenapi.py index 30e75d95a..f6ff23aba 100644 --- a/nova/tests/virt/xenapi/test_xenapi.py +++ b/nova/tests/virt/xenapi/test_xenapi.py @@ -788,11 +788,6 @@ class XenAPIVMTestCase(stubs.XenAPITestBase): Verifies that the VM and VDIs created are properly cleaned up. """ - def fake_hard_shutdown_vm(session, instance, vm_ref): - return True - - self.stubs.Set(vm_utils, 'hard_shutdown_vm', - fake_hard_shutdown_vm) stubs.stubout_attach_disks(self.stubs) vdi_recs_start = self._list_vdis() start_vms = self._list_vms() diff --git a/nova/virt/xenapi/vmops.py b/nova/virt/xenapi/vmops.py index d18fc2729..80a4fb48a 100644 --- a/nova/virt/xenapi/vmops.py +++ b/nova/virt/xenapi/vmops.py @@ -1253,11 +1253,7 @@ class VMOps(object): instance=instance) return - shutdown_success = vm_utils.hard_shutdown_vm(self._session, instance, - vm_ref) - if not shutdown_success: - raise exception.InstancePowerOffFailure( - _("XenAPI failed to power the instance off")) + vm_utils.hard_shutdown_vm(self._session, instance, vm_ref) if destroy_disks: self._volumeops.detach_all(vm_ref) |