diff options
| author | Josh Kearney <josh@jk0.org> | 2011-03-11 11:22:23 -0600 |
|---|---|---|
| committer | Josh Kearney <josh@jk0.org> | 2011-03-11 11:22:23 -0600 |
| commit | 36b5f7d9cf377ce2a4dcdad07e7e14062cd3ec4d (patch) | |
| tree | 1b14c283f17ffda0b6a988d160931219dc43a22c | |
| parent | 4ead485ab69ee1e92635857ba73133a9e1d3bbcb (diff) | |
| download | nova-36b5f7d9cf377ce2a4dcdad07e7e14062cd3ec4d.tar.gz nova-36b5f7d9cf377ce2a4dcdad07e7e14062cd3ec4d.tar.xz nova-36b5f7d9cf377ce2a4dcdad07e7e14062cd3ec4d.zip | |
Further vmops cleanup
| -rw-r--r-- | nova/virt/xenapi/vmops.py | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/nova/virt/xenapi/vmops.py b/nova/virt/xenapi/vmops.py index 5375df5b4..f012fa446 100644 --- a/nova/virt/xenapi/vmops.py +++ b/nova/virt/xenapi/vmops.py @@ -536,10 +536,10 @@ class VMOps(object): """ instance_id = instance.id LOG.info(_("Destroying VM for Instance %(instance_id)s") % locals()) - vm = VMHelper.lookup(self._session, instance.name) - return self._destroy(instance, vm, shutdown=True) + vm_ref = VMHelper.lookup(self._session, instance.name) + return self._destroy(instance, vm_ref, shutdown=True) - def _destroy(self, instance, vm, shutdown=True, + def _destroy(self, instance, vm_ref, shutdown=True, destroy_kernel_ramdisk=True): """ Destroys VM instance by performing: @@ -549,17 +549,17 @@ class VMOps(object): 3. Destroying kernel and ramdisk files (if necessary) 4. Destroying that actual VM record """ - if vm is None: + if vm_ref is None: LOG.warning(_("VM is not present, skipping destroy...")) return if shutdown: - self._shutdown(instance, vm) + self._shutdown(instance, vm_ref) - self._destroy_vdis(instance, vm) + self._destroy_vdis(instance, vm_ref) if destroy_kernel_ramdisk: - self._destroy_kernel_ramdisk(instance, vm) - self._destroy_vm(instance, vm) + self._destroy_kernel_ramdisk(instance, vm_ref) + self._destroy_vm(instance, vm_ref) def _wait_with_callback(self, instance_id, task, callback): ret = None |
