Re: Review Request 25885: CLOUDSTACK-7594: Adding automation test cases for Stopped VM test path

2015-02-20 Thread SrikanteswaraRao Talluri 

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

Ship it!


04409609336f3aee1bc91c875dd3c3badcc71ae5 master

- SrikanteswaraRao Talluri


On Feb. 16, 2015, 5:16 a.m., Ashutosh Kelkar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25885/
> ---
> 
> (Updated Feb. 16, 2015, 5:16 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 
> 
> 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
> 
>

Re: Review Request 25885: CLOUDSTACK-7594: Adding automation test cases for Stopped VM test path

2015-02-19 Thread suresh sadhu 

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

Ship it!


Ship It!

- suresh sadhu


On Feb. 16, 2015, 5:16 a.m., Ashutosh Kelkar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25885/
> ---
> 
> (Updated Feb. 16, 2015, 5:16 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 
> 
> 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
> 
>

Re: Review Request 25885: CLOUDSTACK-7594: Adding automation test cases for Stopped VM test path

2015-02-15 Thread Ashutosh Kelkar 

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


As discussed, removed the code snippet, and all the test cases are passing.
Please review and mark it as ship.

- Ashutosh Kelkar


On Feb. 16, 2015, 5:16 a.m., Ashutosh Kelkar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25885/
> ---
> 
> (Updated Feb. 16, 2015, 5:16 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 
> 
> 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
> 
>

Re: Review Request 25885: CLOUDSTACK-7594: Adding automation test cases for Stopped VM test path

2015-02-15 Thread Ashutosh Kelkar 

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

(Updated Feb. 16, 2015, 5:16 a.m.)


Review request for cloudstack, suresh sadhu and SrikanteswaraRao Talluri.


Changes
---

Review changes and rebase patch against latest master.


Bugs: CLOUDSTACK-7594
https://issues.apache.org/jira/browse/CLOUDSTACK-7594


Repository: cloudstack-git


Description
---

Automation test cases for Stopped VM test path.


Diffs (updated)
-

  test/integration/testpaths/testpath_stopped_vm.py PRE-CREATION 

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

Re: Review Request 25885: CLOUDSTACK-7594: Adding automation test cases for Stopped VM test path

2015-02-15 Thread Ashutosh Kelkar 


> On Feb. 4, 2015, 10:48 a.m., suresh sadhu wrote:
> > you didn't remove the below code... as its not required  in test03 -- here 
> > vm deployed in stopped state..
> > 
> > if str(self.zone.networktype).lower() == "advanced":
> > response = VerifyRouterState(
> > self.apiclient,
> > self.account.name,
> > self.account.domainid,
> > STOPPED
> > )

I have added new patch removing the code snippet and also rebased it against 
latest master branch.


- Ashutosh


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


On Feb. 16, 2015, 5:16 a.m., Ashutosh Kelkar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25885/
> ---
> 
> (Updated Feb. 16, 2015, 5:16 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 
> 
> 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
> 
>

Re: Review Request 25885: CLOUDSTACK-7594: Adding automation test cases for Stopped VM test path

2015-02-04 Thread suresh sadhu 

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


you didn't remove the below code... as its not required  in test03 -- here vm 
deployed in stopped state..

if str(self.zone.networktype).lower() == "advanced":
response = VerifyRouterState(
self.apiclient,
self.account.name,
self.account.domainid,
STOPPED
)

- suresh sadhu


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

RE: Review Request 25885: CLOUDSTACK-7594: Adding automation test cases for Stopped VM test path

2015-02-03 Thread Suresh Sadhu 
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
>
>

Re: Review Request 25885: CLOUDSTACK-7594: Adding automation test cases for Stopped VM test path

2015-02-03 Thread Ashutosh Kelkar 

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

Re: Review Request 25885: CLOUDSTACK-7594: Adding automation test cases for Stopped VM test path

2015-01-22 Thread Ashutosh Kelkar 

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



test/integration/testpaths/testpath_stopped_vm.py


2nd test verifies SSH to VM, while this test is targetted to check the 
router state.


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

Re: Review Request 25885: CLOUDSTACK-7594: Adding automation test cases for Stopped VM test path

2014-10-06 Thread suresh sadhu 

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


def test_03_pt_deploy_vm_with_startvm_false(self):

in test03,why are we checking for routervm stop state when we are deploying vm 
in stopped state.

- suresh sadhu


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

Re: Review Request 25885: CLOUDSTACK-7594: Adding automation test cases for Stopped VM test path

2014-10-01 Thread Ashutosh Kelkar 

---
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 (updated)
-

  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 (updated)
---

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

Re: Review Request 25885: CLOUDSTACK-7594: Adding automation test cases for Stopped VM test path

2014-09-29 Thread SrikanteswaraRao Talluri 

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



test/integration/testpaths/testpath_stopped_vm.py


Did you test this?, Router won't be destroyed immediately. It will happen 
after gc interval.



test/integration/testpaths/testpath_stopped_vm.py


same as previous comment, Router won't be destroyed immediately after 
destroying all the guests in that network.



test/integration/testpaths/testpath_stopped_vm.py


What is the difference between 2nd test and this?


- SrikanteswaraRao Talluri


On Sept. 22, 2014, 5:46 a.m., Ashutosh Kelkar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25885/
> ---
> 
> (Updated Sept. 22, 2014, 5:46 a.m.)
> 
> 
> Review request for cloudstack 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/__init__.py PRE-CREATION 
>   test/integration/testpaths/testpath_stopped_vm.py PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/25885/diff/
> 
> 
> Testing
> ---
> 
> Yes. 4 test cases are failing due to wrong router state, I am investigating 
> on this.
> 
> Log:
> Positive test for stopped VM test path - T1 ... === TestName: 
> test_01_pt_deploy_vm_without_startvm | Status : FAILED ===
> FAIL
> Positive test for stopped VM test path - T1 variant ... === TestName: 
> test_02_pt_deploy_vm_with_startvm_true | Status : FAILED ===
> FAIL
> Positive test for stopped VM test path - T2 ... === TestName: 
> test_03_pt_deploy_vm_with_startvm_false | Status : FAILED ===
> FAIL
> Positive test for stopped VM test path - T2 variant ... === TestName: 
> test_04_pt_deploy_vm_with_startvm_true | Status : FAILED ===
> FAIL
> Positive test for stopped VM test path - T3 and variant ... === TestName: 
> test_05_pt_startvm_false_attach_disk | Status : SUCCESS ===
> ok
> Positive test for stopped VM test path - T4 ... === TestName: 
> test_06_pt_startvm_false_attach_disk_change_SO | Status : SUCCESS ===
> ok
> Positive test for stopped VM test path - T5 ... === TestName: 
> test_07_pt_startvm_false_attach_iso | Status : SUCCESS ===
> ok
> Positive test for stopped VM test path - T5 variant ... === TestName: 
> test_08_pt_startvm_false_attach_iso_running_vm | Status : SUCCESS ===
> ok
> 
> ==
> FAIL: Positive test for stopped VM test path - T1
> --
> Traceback (most recent call last):
>   File "/root/test/testpath_stopped_vm.py", line 260, in 
> test_01_pt_deploy_vm_without_startvm
> self.assertTrue(response[0], response[1])
> AssertionError: Router state should be Stopped, it is Running
>  >> begin captured stdout << -
> === TestName: test_01_pt_deploy_vm_without_startvm | Status : FAILED ===
> 
> 
> Thanks,
> 
> Ashutosh Kelkar
> 
>

Re: Review Request 25885: CLOUDSTACK-7594: Adding automation test cases for Stopped VM test path

2014-09-28 Thread SrikanteswaraRao Talluri 

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


include Suresh Sadhu as a Reviewer.

- SrikanteswaraRao Talluri


On Sept. 22, 2014, 5:46 a.m., Ashutosh Kelkar wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/25885/
> ---
> 
> (Updated Sept. 22, 2014, 5:46 a.m.)
> 
> 
> Review request for cloudstack 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/__init__.py PRE-CREATION 
>   test/integration/testpaths/testpath_stopped_vm.py PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/25885/diff/
> 
> 
> Testing
> ---
> 
> Yes. 4 test cases are failing due to wrong router state, I am investigating 
> on this.
> 
> Log:
> Positive test for stopped VM test path - T1 ... === TestName: 
> test_01_pt_deploy_vm_without_startvm | Status : FAILED ===
> FAIL
> Positive test for stopped VM test path - T1 variant ... === TestName: 
> test_02_pt_deploy_vm_with_startvm_true | Status : FAILED ===
> FAIL
> Positive test for stopped VM test path - T2 ... === TestName: 
> test_03_pt_deploy_vm_with_startvm_false | Status : FAILED ===
> FAIL
> Positive test for stopped VM test path - T2 variant ... === TestName: 
> test_04_pt_deploy_vm_with_startvm_true | Status : FAILED ===
> FAIL
> Positive test for stopped VM test path - T3 and variant ... === TestName: 
> test_05_pt_startvm_false_attach_disk | Status : SUCCESS ===
> ok
> Positive test for stopped VM test path - T4 ... === TestName: 
> test_06_pt_startvm_false_attach_disk_change_SO | Status : SUCCESS ===
> ok
> Positive test for stopped VM test path - T5 ... === TestName: 
> test_07_pt_startvm_false_attach_iso | Status : SUCCESS ===
> ok
> Positive test for stopped VM test path - T5 variant ... === TestName: 
> test_08_pt_startvm_false_attach_iso_running_vm | Status : SUCCESS ===
> ok
> 
> ==
> FAIL: Positive test for stopped VM test path - T1
> --
> Traceback (most recent call last):
>   File "/root/test/testpath_stopped_vm.py", line 260, in 
> test_01_pt_deploy_vm_without_startvm
> self.assertTrue(response[0], response[1])
> AssertionError: Router state should be Stopped, it is Running
>  >> begin captured stdout << -
> === TestName: test_01_pt_deploy_vm_without_startvm | Status : FAILED ===
> 
> 
> Thanks,
> 
> Ashutosh Kelkar
> 
>

Review Request 25885: CLOUDSTACK-7594: Adding automation test cases for Stopped VM test path

2014-09-21 Thread Ashutosh Kelkar 

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

Review request for cloudstack 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/__init__.py PRE-CREATION 
  test/integration/testpaths/testpath_stopped_vm.py PRE-CREATION 

Diff: https://reviews.apache.org/r/25885/diff/


Testing
---

Yes. 4 test cases are failing due to wrong router state, I am investigating on 
this.

Log:
Positive test for stopped VM test path - T1 ... === TestName: 
test_01_pt_deploy_vm_without_startvm | Status : FAILED ===
FAIL
Positive test for stopped VM test path - T1 variant ... === TestName: 
test_02_pt_deploy_vm_with_startvm_true | Status : FAILED ===
FAIL
Positive test for stopped VM test path - T2 ... === TestName: 
test_03_pt_deploy_vm_with_startvm_false | Status : FAILED ===
FAIL
Positive test for stopped VM test path - T2 variant ... === TestName: 
test_04_pt_deploy_vm_with_startvm_true | Status : FAILED ===
FAIL
Positive test for stopped VM test path - T3 and variant ... === TestName: 
test_05_pt_startvm_false_attach_disk | Status : SUCCESS ===
ok
Positive test for stopped VM test path - T4 ... === TestName: 
test_06_pt_startvm_false_attach_disk_change_SO | Status : SUCCESS ===
ok
Positive test for stopped VM test path - T5 ... === TestName: 
test_07_pt_startvm_false_attach_iso | Status : SUCCESS ===
ok
Positive test for stopped VM test path - T5 variant ... === TestName: 
test_08_pt_startvm_false_attach_iso_running_vm | Status : SUCCESS ===
ok

==
FAIL: Positive test for stopped VM test path - T1
--
Traceback (most recent call last):
  File "/root/test/testpath_stopped_vm.py", line 260, in 
test_01_pt_deploy_vm_without_startvm
self.assertTrue(response[0], response[1])
AssertionError: Router state should be Stopped, it is Running
 >> begin captured stdout << -
=== TestName: test_01_pt_deploy_vm_without_startvm | Status : FAILED ===


Thanks,

Ashutosh Kelkar