----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14199/#review26715 -----------------------------------------------------------
test/integration/component/test_snapshots_improvement.py <https://reviews.apache.org/r/14199/#comment52037> Do check the state of the snapshot as well. test/integration/component/test_snapshots_improvement.py <https://reviews.apache.org/r/14199/#comment52045> step 6 is happening before step 5 - Nitin Mehta On Sept. 18, 2013, 11:25 a.m., Gaurav Aradhye wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/14199/ > ----------------------------------------------------------- > > (Updated Sept. 18, 2013, 11:25 a.m.) > > > Review request for cloudstack and Prasanna Santhanam. > > > Repository: cloudstack-git > > > Description > ------- > > Adding 5 missing test cases. > Change in asyncJobMgr.py >> the method "make_request" in cloudstackConnection > does not exist now. Replaced it by "marvin_request". Checked running async > jobs with this change. Works correctly. > > > Diffs > ----- > > test/integration/component/test_snapshots_improvement.py PRE-CREATION > tools/marvin/marvin/asyncJobMgr.py 25818a6 > > Diff: https://reviews.apache.org/r/14199/diff/ > > > Testing > ------- > > Tested locally. > > > Thanks, > > Gaurav Aradhye > >