----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13469/#review24985 -----------------------------------------------------------
Ship it! Ship It! - Devdeep Singh 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 > >
