> 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 > >
