> On July 13, 2013, 4:53 a.m., Prasanna Santhanam wrote:
> > server/src/com/cloud/storage/VolumeManagerImpl.java, line 1124
> > <https://reviews.apache.org/r/12521/diff/1/?file=320928#file320928line1124>
> >
> >     Well - this comment should have gone away ;)

Ok - I'll update that thread on the list. I see your comments on the bug. 
Thanks.

The commit for 9241 is here

commit c5185c09efd8747f22e1ed4268cea3d037d6fde6
Author: Marcus Sorensen <mar...@betterservers.com>
Date:   Wed Feb 6 07:52:37 2013 -0700

    Summary: Add missing ResizeVolumeCommand in VmwareResource.java
    
    Submitted-by: hongtu_zang <hongtu_z...@tcloudcomputing.com>
    Signed-off-by: Marcus Sorensen <mar...@betterservers.com>


- Prasanna


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


On July 12, 2013, 9:02 p.m., Venkata Siva Vijayendra Bhamidipati wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12521/
> -----------------------------------------------------------
> 
> (Updated July 12, 2013, 9:02 p.m.)
> 
> 
> Review request for cloudstack and edison su.
> 
> 
> Bugs: CLOUDSTACK-3509
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> Put in vmware in a check filtering hypervisors supporting volume resize 
> operations.
> 
> 
> Diffs
> -----
> 
>   server/src/com/cloud/storage/VolumeManagerImpl.java 1618574 
> 
> Diff: https://reviews.apache.org/r/12521/diff/
> 
> 
> Testing
> -------
> 
> Verified that volume resize of a non root volume of a guest VM in a vmware 
> deployment works instead of throwing the exception pasted in the bug 
> description.
> 
> 
> Thanks,
> 
> Venkata Siva Vijayendra Bhamidipati
> 
>

Reply via email to