cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF Subversion and Git Services" <asf...@urd.zones.apache.org>
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:20:29 GMT

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


Commit be9dbcc2c5edb995c48558ea5e1a87c8d3012965 in branch refs/heads/master from Rajesh Battala
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=be9dbcc ]

CLOUDSTACK-3946 [KVM] [Zone Wide Primary Storages] migrateVolume is deleting the volume if
the source and destination pool are same


- ASF Subversion and Git Services


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