Repository: cloudstack Updated Branches: refs/heads/master e2b13a344 -> 4fcd0b1a1
if libvirt fails to attach disks, call disconnectPhysicalDisk to clean up Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/4fcd0b1a Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/4fcd0b1a Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/4fcd0b1a Branch: refs/heads/master Commit: 4fcd0b1a1f853882c38ac13a618a8264051e1af5 Parents: e2b13a3 Author: Marcus Sorensen <[email protected]> Authored: Thu Feb 20 15:43:27 2014 -0700 Committer: Marcus Sorensen <[email protected]> Committed: Thu Feb 20 15:43:27 2014 -0700 ---------------------------------------------------------------------- .../src/com/cloud/hypervisor/kvm/storage/KVMStorageProcessor.java | 1 + 1 file changed, 1 insertion(+) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/cloudstack/blob/4fcd0b1a/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/KVMStorageProcessor.java ---------------------------------------------------------------------- diff --git a/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/KVMStorageProcessor.java b/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/KVMStorageProcessor.java index 1c37607..50671fa 100644 --- a/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/KVMStorageProcessor.java +++ b/plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/KVMStorageProcessor.java @@ -971,6 +971,7 @@ public class KVMStorageProcessor implements StorageProcessor { return new AttachAnswer(disk); } catch (LibvirtException e) { s_logger.debug("Failed to attach volume: " + vol.getPath() + ", due to " + e.toString()); + storagePoolMgr.disconnectPhysicalDisk(primaryStore.getPoolType(), primaryStore.getUuid(), vol.getPath()); return new AttachAnswer(e.toString()); } catch (InternalErrorException e) { s_logger.debug("Failed to attach volume: " + vol.getPath() + ", due to " + e.toString());
