cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ashutosh Kelkar" <ashut...@clogeny.com>
Subject Re: Review Request 17702: CLOUDSTACK-2232: Adding automation test cases for Persistent Networks feature
Date Fri, 14 Feb 2014 10:45:27 GMT


> On Feb. 12, 2014, 5:23 p.m., Santhosh Edukulla wrote:
> > test/integration/component/test_persistent_networks.py, line 1336
> > <https://reviews.apache.org/r/17702/diff/1/?file=468548#file468548line1336>
> >
> >     Is waiting required post disable\lock? or after Account.list?

Yes, because we have to check the "persistent" property of network, for that we will have
to wait for cleanup interval post the lock/disable action. We will not catch the bug if we
don't wait, because network won't get cleaned up immediately.


- Ashutosh


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


On Feb. 14, 2014, 10:38 a.m., Ashutosh Kelkar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/17702/
> -----------------------------------------------------------
> 
> (Updated Feb. 14, 2014, 10:38 a.m.)
> 
> 
> Review request for cloudstack, Girish Shilamkar and Santhosh Edukulla.
> 
> 
> Bugs: CLOUDSTACK-2232
>     https://issues.apache.org/jira/browse/CLOUDSTACK-2232
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> Adding automation test cases for Persistent Networks feature.
> 
> 
> Diffs
> -----
> 
>   test/integration/component/test_persistent_networks.py f61ccaa 
>   tools/marvin/marvin/config/config.cfg 7840b7c 
>   tools/marvin/marvin/integration/lib/base.py aa23029 
>   tools/marvin/marvin/integration/lib/common.py e202391 
> 
> Diff: https://reviews.apache.org/r/17702/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Ashutosh Kelkar
> 
>


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