Sure,I will review it and will let you know after checking the details . Thanks sadhu ________________________________________ From: Ashutosh Kelkar [nore...@reviews.apache.org] on behalf of Ashutosh Kelkar [ashut...@clogeny.com] Sent: Wednesday, February 04, 2015 11:26 AM To: suresh sadhu; Srikanteswararao Talluri Cc: Ashutosh Kelkar; cloudstack Subject: Re: Review Request 25885: CLOUDSTACK-7594: Adding automation test cases for Stopped VM test path
----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25885/#review70927 ----------------------------------------------------------- I have addressed all the review comments, and also verified that patch is still valid against latest master branch. It applies cleanly. Sadhu, can you please look at this? - Ashutosh Kelkar On Oct. 1, 2014, 9:18 a.m., Ashutosh Kelkar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/25885/ > ----------------------------------------------------------- > > (Updated Oct. 1, 2014, 9:18 a.m.) > > > Review request for cloudstack, suresh sadhu and SrikanteswaraRao Talluri. > > > Bugs: CLOUDSTACK-7594 > https://issues.apache.org/jira/browse/CLOUDSTACK-7594 > > > Repository: cloudstack-git > > > Description > ------- > > Automation test cases for Stopped VM test path. > > > Diffs > ----- > > test/integration/testpaths/testpath_stopped_vm.py PRE-CREATION > tools/marvin/marvin/lib/base.py d623386 > > Diff: https://reviews.apache.org/r/25885/diff/ > > > Testing > ------- > > Yes. > > Advanced zone: > Positive test for stopped VM test path - T1 ... === TestName: > test_01_pt_deploy_vm_without_startvm | Status : SUCCESS === > ok > Positive test for stopped VM test path - T1 variant ... === TestName: > test_02_pt_deploy_vm_with_startvm_true | Status : SUCCESS === > ok > Positive test for stopped VM test path - T2 ... === TestName: > test_03_pt_deploy_vm_with_startvm_false | Status : SUCCESS === > ok > Positive test for stopped VM test path - T3 and variant, T9 ... === TestName: > test_04_pt_startvm_false_attach_disk | Status : SUCCESS === > ok > Positive test for stopped VM test path - T4 ... === TestName: > test_05_pt_startvm_false_attach_disk_change_SO | Status : SUCCESS === > ok > Positive test for stopped VM test path - T5 ... === TestName: > test_06_pt_startvm_false_attach_iso | Status : SUCCESS === > ok > Positive test for stopped VM test path - T5 variant ... === TestName: > test_07_pt_startvm_false_attach_iso_running_vm | Status : SUCCESS === > ok > Positive test for stopped VM test path - T10 ... === TestName: > test_08_pt_startvm_false_password_enabled_template | Status : SUCCESS === > ok > Positive test for stopped VM test path - T11 ... === TestName: > test_09_pt_destroy_stopped_vm | Status : SUCCESS === > ok > Positive test for stopped VM test path - T12 ... === TestName: > test_10_max_account_limit | Status : SUCCESS === > ok > > > Basic zone: > > Positive test for stopped VM test path - T1 ... === TestName: > test_01_pt_deploy_vm_without_startvm | Status : SUCCESS === > ok > Positive test for stopped VM test path - T1 variant ... === TestName: > test_02_pt_deploy_vm_with_startvm_true | Status : SUCCESS === > ok > Positive test for stopped VM test path - T2 ... === TestName: > test_03_pt_deploy_vm_with_startvm_false | Status : SUCCESS === > ok > Positive test for stopped VM test path - T3 and variant, T9 ... === TestName: > test_04_pt_startvm_false_attach_disk | Status : SUCCESS === > ok > Positive test for stopped VM test path - T4 ... === TestName: > test_05_pt_startvm_false_attach_disk_change_SO | Status : SUCCESS === > ok > Positive test for stopped VM test path - T5 ... === TestName: > test_06_pt_startvm_false_attach_iso | Status : SUCCESS === > ok > Positive test for stopped VM test path - T5 variant ... === TestName: > test_07_pt_startvm_false_attach_iso_running_vm | Status : SUCCESS === > ok > Positive test for stopped VM test path - T10 ... === TestName: > test_08_pt_startvm_false_password_enabled_template | Status : SUCCESS === > ok > Positive test for stopped VM test path - T11 ... === TestName: > test_09_pt_destroy_stopped_vm | Status : SUCCESS === > ok > Positive test for stopped VM test path - T12 ... === TestName: > test_10_max_account_limit | Status : SUCCESS === > ok > > ---------------------------------------------------------------------- > Ran 10 tests in 2792.968s > > OK > > > Thanks, > > Ashutosh Kelkar > >
