----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10539/ -----------------------------------------------------------
(Updated April 19, 2013, 9:34 a.m.) Review request for cloudstack and edison su. Changes ------- Updated the patch after merging with the latest changes on master. Description ------- The state transitions on the volume that has to be migrated were incorrect. A volume to be migrated is in ready state and cannot be transitioned to copying state. Similarly, the duplicated volume is in ready state too and cannot be transitioned to creating state. Fixed it by transitioning the volume to migrating state when a migrateVolume api call is made. Also, a volume has to be destroyed first before it can be expunged. Fixed that too. This addresses bug https://issues.apache.org/jira/browse/CLOUDSTACK-1879. Diffs (updated) ----- engine/storage/volume/src/org/apache/cloudstack/storage/volume/VolumeServiceImpl.java e3526de Diff: https://reviews.apache.org/r/10539/diff/ Testing ------- 1. migrated a volume from one storage pool to another and made sure it was successful. Attached it to a vm after migrating. 2. Also verified that the patch applies successfully. Thanks, Devdeep Singh
