You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Santhosh Edukulla <sa...@citrix.com> on 2014/05/07 16:57:57 UTC

Review Request 21161: Fixed few cases failing, added proper port for creating client

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

Review request for cloudstack and Girish Shilamkar.


Repository: cloudstack-git


Description
-------

1. Fied few cases failing earlier because of wrong zone,domain retrieval
2. Added proper port usage while creating client


Diffs
-----

  test/integration/smoke/test_deploy_vm.py c07e663 
  tools/marvin/marvin/cloudstackTestClient.py 0e3d3d0 
  tools/marvin/marvin/marvinPlugin.py 74b64ef 

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


Testing
-------

yes, done the changes and tested on simulator

Test Deploy Virtual Machine ... === TestName: test_deploy_vm | Status : SUCCESS ===
ok
Test Multiple Deploy Virtual Machine ... === TestName: test_deploy_vm_multiple | Status : SUCCESS ===
ok
Test Deploy Virtual Machine - start operation failure and retry ... === TestName: test_deploy_vm_start_failure | Status : SUCCESS ===
ok
Test Deploy Virtual Machine - volume creation failure and retry ... === TestName: test_deploy_vm_volume_creation_failure | Status : SUCCESS ===
ok

----------------------------------------------------------------------
Ran 4 tests in 138.006s

OK
~                                                                                                               
~       


Thanks,

Santhosh Edukulla


Re: Review Request 21161: Fixed few cases failing, added proper port for creating client

Posted by Jayapal Reddy <ja...@citrix.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/21161/#review42494
-----------------------------------------------------------

Ship it!


Ship It!

- Jayapal Reddy


On May 7, 2014, 2:57 p.m., Santhosh Edukulla wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/21161/
> -----------------------------------------------------------
> 
> (Updated May 7, 2014, 2:57 p.m.)
> 
> 
> Review request for cloudstack and Girish Shilamkar.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> 1. Fied few cases failing earlier because of wrong zone,domain retrieval
> 2. Added proper port usage while creating client
> 
> 
> Diffs
> -----
> 
>   test/integration/smoke/test_deploy_vm.py c07e663 
>   tools/marvin/marvin/cloudstackTestClient.py 0e3d3d0 
>   tools/marvin/marvin/marvinPlugin.py 74b64ef 
> 
> Diff: https://reviews.apache.org/r/21161/diff/
> 
> 
> Testing
> -------
> 
> yes, done the changes and tested on simulator
> 
> Test Deploy Virtual Machine ... === TestName: test_deploy_vm | Status : SUCCESS ===
> ok
> Test Multiple Deploy Virtual Machine ... === TestName: test_deploy_vm_multiple | Status : SUCCESS ===
> ok
> Test Deploy Virtual Machine - start operation failure and retry ... === TestName: test_deploy_vm_start_failure | Status : SUCCESS ===
> ok
> Test Deploy Virtual Machine - volume creation failure and retry ... === TestName: test_deploy_vm_volume_creation_failure | Status : SUCCESS ===
> ok
> 
> ----------------------------------------------------------------------
> Ran 4 tests in 138.006s
> 
> OK
> ~                                                                                                               
> ~       
> 
> 
> Thanks,
> 
> Santhosh Edukulla
> 
>