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

Ship it!


Ship It!

- Devdeep Singh


On May 9, 2013, 12:49 p.m., Rajesh Battala wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/11022/
> -----------------------------------------------------------
> 
> (Updated May 9, 2013, 12:49 p.m.)
> 
> 
> Review request for cloudstack, Devdeep Singh and edison su.
> 
> 
> Description
> -------
> 
> When user uploads the volume, successful uploaded volume state in DB is still 
> marked as "UploadOp"
> 
> This is issue with the VolumeServiceImpl, after uploaded operation is 
> successful it should call the stateTransit  "UploadOp" to "Uploaded".
> Issue is fixed by call the volume state change and now the volume is moved to 
> "uploaded" state which will not throw the exception as the volume is not in 
> ready state.
> 
> 
> This addresses bug CLOUDSTACK-2081.
> 
> 
> Diffs
> -----
> 
>   
> engine/storage/volume/src/org/apache/cloudstack/storage/volume/VolumeServiceImpl.java
>  b39502b 
> 
> Diff: https://reviews.apache.org/r/11022/diff/
> 
> 
> Testing
> -------
> 
> Unittests are running successful.
> 
> 1. Uploaded the volume, verified the state column in db. during uploading 
> progress its is "UploadOp" after upload is successful it is moved to 
> "Uploaded"
> 
> 
> Thanks,
> 
> Rajesh Battala
> 
>

Reply via email to