cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Rajesh Battala" <rajesh.batt...@citrix.com>
Subject Re: Review Request 13469: CLOUDSTACK-3946 [KVM] [Zone Wide Primary Storages] migrateVolume is deleting the volume if the source and destination pool are same.
Date Mon, 12 Aug 2013 08:46:51 GMT


> On Aug. 12, 2013, 8:25 a.m., Devdeep Singh wrote:
> > Ship It!

Thanks for reviewing and pushing the changes.


- Rajesh


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


On Aug. 12, 2013, 6:48 a.m., Rajesh Battala wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13469/
> -----------------------------------------------------------
> 
> (Updated Aug. 12, 2013, 6:48 a.m.)
> 
> 
> Review request for cloudstack, Devdeep Singh, edison su, and Ram Ganesh.
> 
> 
> Bugs: CLOUDSTACK-3946
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> while migrating volume, if the storage and destination pool are same, we are still do
overhead of copying the volume to same pool and deleting the other src copy.
> 
> fixed the issue by restricting volumemigration if the source and destination pool are
same.
> 
> 
> Diffs
> -----
> 
>   server/src/com/cloud/storage/VolumeManagerImpl.java 100aad3 
> 
> Diff: https://reviews.apache.org/r/13469/diff/
> 
> 
> Testing
> -------
> 
> 1. Successfully migrated the volume from Storagepool A to pool B.
> 2. migration is not allowed if the source and destination are same.
> 3. tried the volume with storage pool scope (ZONE and Cluster)
> 
> 
> Thanks,
> 
> Rajesh Battala
> 
>


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