-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/7459/
-----------------------------------------------------------

Review request for cloudstack.


Description
-------

When a host has maintenance cancelled, it runs a StopCommand for every instance 
that was migrated off prior to enabling maintenance. This can delete patch 
disks for the system VMs while they're running on other hosts.  This patch 
keeps us from deleting patch disks via StopCommand (partially reverting back to 
3.0.x behavior).


Diffs
-----

  
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java
 22b149f 

Diff: https://reviews.apache.org/r/7459/diff/


Testing
-------

Previously saw this for every instance previously running on host when 
executing cancel maintenance:

2012-10-05 16:01:24,510 DEBUG [cloud.agent.Agent] (agentRequest-Handler-1:null) 
Processing command: com.cloud.agent.api.StopCommand
2012-10-05 16:01:24,795 DEBUG [kvm.storage.LibvirtStorageAdaptor] 
(agentRequest-Handler-1:null) requested delete disk /dev/vg0/r-37-VM-patchdisk

Post patch when I cancel maintenance, I see the stop command but no 'requested 
delete disk' for patchdisk.


Thanks,

Marcus Sorensen

Reply via email to