Interesting...that commit was a combination of Marcus' and my code. We might want to bring him in on this conversation.
I believe the idea is when a VM is stopped that we want to remove, for example, its iSCSI connections to its data disks (just using iSCSI as an example here). This is similar (but not the same of course) to how we disconnect a VBD from a VDI for a VM when the VM is stopped. When the VM is restarted, we create a new VBD and connect the VDI to the VM through it (and there is an analogous process that LibvirtStorageAdaptor does during VM start). On Thu, Dec 19, 2013 at 12:08 PM, Edison Su <edison...@citrix.com> wrote: > Hi Mike, > > I looked at your commit: 858ce766659101eb731c83c806892dd5d9baa976, > seems it will try to delete primary storage every time when stopping a VM > ,which maybe the root cause a kvm blocker bug: CLOUDSTACK-5432, KVM guest > vms are crashed during the automation test. From the agent log, I find a > lot of “umount ” primary storage error. > > In 4.2, we never do that kind of operation. Do you know why we add it in > 4.3? I am referring to the code in : > > > > disconnectPhysicalDiskByPath in LibvirtStorageAdaptor > -- *Mike Tutkowski* *Senior CloudStack Developer, SolidFire Inc.* e: mike.tutkow...@solidfire.com o: 303.746.7302 Advancing the way the world uses the cloud<http://solidfire.com/solution/overview/?video=play> *™*