cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF Subversion and Git Services" <asf...@urd.zones.apache.org>
Subject Re: Review Request 17447: Fix for CLOUDSTACK-5948
Date Wed, 29 Jan 2014 05:07:38 GMT

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


Commit d057585128066230a782b66fe306ef536eb5f259 in branch refs/heads/marvin from Santhosh
Edukulla
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=d057585 ]

CLOUDSTACK-5948: Fixes to BVT test_volumes.py

1. Added a generic way of verifying isk volumes.
2. Now, detach volume is called post the assert.
3. Added a sleep post detach volume to facilitate list volumes to have some time for detach
operation.
4. Added a fix for a log check.


- ASF Subversion and Git Services


On Jan. 28, 2014, 8:34 a.m., Santhosh Edukulla wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/17447/
> -----------------------------------------------------------
> 
> (Updated Jan. 28, 2014, 8:34 a.m.)
> 
> 
> Review request for cloudstack and Girish Shilamkar.
> 
> 
> Bugs: CLOUDSTACK-5948
>     https://issues.apache.org/jira/browse/CLOUDSTACK-5948
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> Fixed the issue 5948
> 1. Added a generic way of verifying isk volumes.
> 2. Now, detach volume is called post the assert.
> 3. Added a sleep post detach volume to facilitate list volumes to have some time for
detach operation.
> 4. Added a fix for a log check.
> 
> 
> Diffs
> -----
> 
>   test/integration/smoke/test_volumes.py 8f41803 
>   tools/marvin/marvin/integration/lib/utils.py e870158 
>   tools/marvin/marvin/marvinInit.py 36d907e 
>   tools/marvin/marvin/sshClient.py fd8726c 
> 
> Diff: https://reviews.apache.org/r/17447/diff/
> 
> 
> Testing
> -------
> 
> 1. Testing on separate topic branch will be done post other fixes.
> 2. Verified the script for any import errors.
> 
> 
> Thanks,
> 
> Santhosh Edukulla
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message