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/02/25 13:24:03 UTC

Review Request 17999: Added few changes for CLOUDSTACK-5674

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

Review request for cloudstack and Girish Shilamkar.


Summary (updated)
-----------------

Added few changes for CLOUDSTACK-5674


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


Repository: cloudstack-git


Description (updated)
-------

Added few changes to fix test suite run and zone default value.


Diffs
-----

  tools/marvin/marvin/asyncJobMgr.py dab85ca 
  tools/marvin/marvin/cloudstackConnection.py b8d9a38 
  tools/marvin/marvin/cloudstackTestClient.py e8c6ab2 
  tools/marvin/marvin/dbConnection.py 422fcfa 
  tools/marvin/marvin/deployDataCenter.py 862a1bf 
  tools/marvin/marvin/lib/utils.py 957807d 
  tools/marvin/marvin/marvinInit.py 5218d23 
  tools/marvin/marvin/marvinLog.py 4161eca 
  tools/marvin/marvin/marvinPlugin.py f36bca8 
  tools/marvin/marvin/sshClient.py 9b5bca2 

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


Testing (updated)
-------

Tested running with multiple parameters.


File Attachments (updated)
----------------

Patch file for fix.
  https://reviews.apache.org/media/uploaded/files/2014/02/25/723e2875-3741-4c57-a70b-1af6dfd474cc__0001-Added-few-changes-for-CLOUDSTACK-5674.patch


Thanks,

Santhosh Edukulla

Re: Review Request 17999: Added few changes for CLOUDSTACK-5674

Posted by ASF Subversion and Git Services <as...@urd.zones.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17999/#review35516
-----------------------------------------------------------


Commit 23e059b1b903bff84155a116432ef02bfd078d9d in cloudstack's branch refs/heads/marvin from Santhosh Edukulla
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=23e059b ]

CLOUDSTACK-5674: Added few changes for CLOUDSTACK-567


- ASF Subversion and Git Services


On Feb. 25, 2014, 12:24 p.m., Santhosh Edukulla wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/17999/
> -----------------------------------------------------------
> 
> (Updated Feb. 25, 2014, 12:24 p.m.)
> 
> 
> Review request for cloudstack and Girish Shilamkar.
> 
> 
> Bugs: CLOUDSTACK-5674
>     https://issues.apache.org/jira/browse/CLOUDSTACK-5674
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> Added few changes to fix test suite run and zone default value.
> 
> 
> Diffs
> -----
> 
>   tools/marvin/marvin/asyncJobMgr.py dab85ca 
>   tools/marvin/marvin/cloudstackConnection.py b8d9a38 
>   tools/marvin/marvin/cloudstackTestClient.py e8c6ab2 
>   tools/marvin/marvin/dbConnection.py 422fcfa 
>   tools/marvin/marvin/deployDataCenter.py 862a1bf 
>   tools/marvin/marvin/lib/utils.py 957807d 
>   tools/marvin/marvin/marvinInit.py 5218d23 
>   tools/marvin/marvin/marvinLog.py 4161eca 
>   tools/marvin/marvin/marvinPlugin.py f36bca8 
>   tools/marvin/marvin/sshClient.py 9b5bca2 
> 
> Diff: https://reviews.apache.org/r/17999/diff/
> 
> 
> Testing
> -------
> 
> Tested running with multiple parameters.
> 
> 
> File Attachments
> ----------------
> 
> Patch file for fix.
>   https://reviews.apache.org/media/uploaded/files/2014/02/25/723e2875-3741-4c57-a70b-1af6dfd474cc__0001-Added-few-changes-for-CLOUDSTACK-5674.patch
> 
> 
> Thanks,
> 
> Santhosh Edukulla
> 
>

Re: Review Request 17999: Added few changes for CLOUDSTACK-5674

Posted by Girish Shilamkar <gi...@clogeny.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17999/#review35517
-----------------------------------------------------------

Ship it!


Ship It!

- Girish Shilamkar


On Feb. 25, 2014, 12:24 p.m., Santhosh Edukulla wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/17999/
> -----------------------------------------------------------
> 
> (Updated Feb. 25, 2014, 12:24 p.m.)
> 
> 
> Review request for cloudstack and Girish Shilamkar.
> 
> 
> Bugs: CLOUDSTACK-5674
>     https://issues.apache.org/jira/browse/CLOUDSTACK-5674
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> Added few changes to fix test suite run and zone default value.
> 
> 
> Diffs
> -----
> 
>   tools/marvin/marvin/asyncJobMgr.py dab85ca 
>   tools/marvin/marvin/cloudstackConnection.py b8d9a38 
>   tools/marvin/marvin/cloudstackTestClient.py e8c6ab2 
>   tools/marvin/marvin/dbConnection.py 422fcfa 
>   tools/marvin/marvin/deployDataCenter.py 862a1bf 
>   tools/marvin/marvin/lib/utils.py 957807d 
>   tools/marvin/marvin/marvinInit.py 5218d23 
>   tools/marvin/marvin/marvinLog.py 4161eca 
>   tools/marvin/marvin/marvinPlugin.py f36bca8 
>   tools/marvin/marvin/sshClient.py 9b5bca2 
> 
> Diff: https://reviews.apache.org/r/17999/diff/
> 
> 
> Testing
> -------
> 
> Tested running with multiple parameters.
> 
> 
> File Attachments
> ----------------
> 
> Patch file for fix.
>   https://reviews.apache.org/media/uploaded/files/2014/02/25/723e2875-3741-4c57-a70b-1af6dfd474cc__0001-Added-few-changes-for-CLOUDSTACK-5674.patch
> 
> 
> Thanks,
> 
> Santhosh Edukulla
> 
>