incubator-cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Nitin Mehta" <nitin.me...@citrix.com>
Subject Re: Review Request: CLOUDSTACK-357 ISOs can be deleted while still attached to a running VM, and they subsequently cannot be detached from a running VM
Date Thu, 21 Feb 2013 08:10:37 GMT

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


Thanks Deepti for the work. Really appreciate the hard work.
Can you please confirm me that this iso wont get deleted during template sync which gets triggered
during stop start ssvm or MS restart ?
Also can you please put in all the test cases u used for my benefit ?

- Nitin Mehta


On Feb. 19, 2013, 5:21 p.m., deepti dohare wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/7660/
> -----------------------------------------------------------
> 
> (Updated Feb. 19, 2013, 5:21 p.m.)
> 
> 
> Review request for cloudstack and Min Chen.
> 
> 
> Description
> -------
> 
> Currently no warning or confirmation is given or required to delete an iso which is attached
to a virtual machine.
> 
> Fixing this issue using “force” delete option.
> If an admin or a user deletes an ISO which is attached to a vm:
> 1. Without force option; gives a warning that ISO is attached to vm “Failed to delete
iso. It is currently attached to virtual machine. Please detach it before deleting”.
> 2. With force option: first detach the ISO from all vms and then deletes the ISO. 
> 
> 
> This addresses bug CLOUDSTACK-357.
> 
> 
> Diffs
> -----
> 
>   server/src/com/cloud/storage/dao/VMTemplateHostDaoImpl.java 4d1ac02 
>   server/src/com/cloud/template/HyervisorTemplateAdapter.java fe6bc2a 
>   server/src/com/cloud/template/TemplateAdapterBase.java fa677ac 
>   server/src/com/cloud/template/TemplateManagerImpl.java f9cf277 
>   server/src/com/cloud/vm/dao/UserVmDao.java 9fbcde3 
>   server/src/com/cloud/vm/dao/UserVmDaoImpl.java f2fc10b 
> 
> Diff: https://reviews.apache.org/r/7660/diff/
> 
> 
> Testing
> -------
> 
> Verified locally. Tested for some cases:
> 
> 1. Deleting iso(not attached to any vm)
> 2. Deleting iso(attached to multiple vms in same domain)
> 3. Deleting iso(attached to multiple vms in different domains)
> 
> For admin/user
> 
> 
> Thanks,
> 
> deepti dohare
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message