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



test/integration/testpaths/testpath_volumelifecycle.py
<https://reviews.apache.org/r/26080/#comment94907>

    Replace with validateList



test/integration/testpaths/testpath_volumelifecycle.py
<https://reviews.apache.org/r/26080/#comment94906>

    Add new line(\n)



test/integration/testpaths/testpath_volumelifecycle.py
<https://reviews.apache.org/r/26080/#comment94904>

    Replace this with validateList



test/integration/testpaths/testpath_volumelifecycle.py
<https://reviews.apache.org/r/26080/#comment94905>

    Replace with assertEqual



test/integration/testpaths/testpath_volumelifecycle.py
<https://reviews.apache.org/r/26080/#comment94902>

    Please remove this line



test/integration/testpaths/testpath_volumelifecycle.py
<https://reviews.apache.org/r/26080/#comment94903>

    Remove the blank lines



test/integration/testpaths/testpath_volumelifecycle.py
<https://reviews.apache.org/r/26080/#comment94908>

    move this line after assertion


- sanjeev n


On Sept. 26, 2014, 10:58 a.m., prashant mishra wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/26080/
> -----------------------------------------------------------
> 
> (Updated Sept. 26, 2014, 10:58 a.m.)
> 
> 
> Review request for cloudstack, sanjeev n and SrikanteswaraRao Talluri.
> 
> 
> Bugs: CLOUDSTACK-7632
>     https://issues.apache.org/jira/browse/CLOUDSTACK-7632
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> Automation for volume life cycle testPath
> 
> 
> Diffs
> -----
> 
>   test/integration/testpaths/testpath_volumelifecycle.py PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/26080/diff/
> 
> 
> Testing
> -------
> 
> tested on following setup 
> 
> 1-only shared storage
> 2-local storage + zone wide storage +shared storage
> 
> 
> Thanks,
> 
> prashant mishra
> 
>

Reply via email to