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 2013/12/11 10:32:49 UTC

Review Request 16177: Fixed CLOUDSTACK-5443

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

Review request for cloudstack and Girish Shilamkar.


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


Repository: cloudstack-git


Description
-------

There was an issue with logging under userTestClient.. test code. More information inside the bug. 
Fixed that.


Diffs
-----

  tools/marvin/marvin/cloudstackConnection.py 48a2d21 
  tools/marvin/marvin/cloudstackTestClient.py 8a41fad 
  tools/marvin/marvin/deployDataCenter.py 0a175d9 

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


Testing
-------

DeployDC and sanity testing worked fine. Ran BVT against simulator


Thanks,

Santhosh Edukulla

Re: Review Request 16177: Fixed CLOUDSTACK-5443

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/16177/#review30195
-----------------------------------------------------------


Commit e498bf00d4c4b781faebabfcd6d0f594f06240d9 in branch refs/heads/master from Santhosh Edukulla
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=e498bf0 ]

CLOUDSTACK-5443: Fixed the issue.


- ASF Subversion and Git Services


On Dec. 11, 2013, 9:32 a.m., Santhosh Edukulla wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/16177/
> -----------------------------------------------------------
> 
> (Updated Dec. 11, 2013, 9:32 a.m.)
> 
> 
> Review request for cloudstack and Girish Shilamkar.
> 
> 
> Bugs: CLOUDSTACK-5443
>     https://issues.apache.org/jira/browse/CLOUDSTACK-5443
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> There was an issue with logging under userTestClient.. test code. More information inside the bug. 
> Fixed that.
> 
> 
> Diffs
> -----
> 
>   tools/marvin/marvin/cloudstackConnection.py 48a2d21 
>   tools/marvin/marvin/cloudstackTestClient.py 8a41fad 
>   tools/marvin/marvin/deployDataCenter.py 0a175d9 
> 
> Diff: https://reviews.apache.org/r/16177/diff/
> 
> 
> Testing
> -------
> 
> DeployDC and sanity testing worked fine. Ran BVT against simulator
> 
> 
> Thanks,
> 
> Santhosh Edukulla
> 
>

Re: Review Request 16177: Fixed CLOUDSTACK-5443

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/16177/#review30197
-----------------------------------------------------------


Commit c494064c0daf7c8d02e10b3f7e670881ede14bd5 in branch refs/heads/4.2 from Santhosh Edukulla
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=c494064 ]

CLOUDSTACK-5443: Fixed the issue.


- ASF Subversion and Git Services


On Dec. 11, 2013, 9:32 a.m., Santhosh Edukulla wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/16177/
> -----------------------------------------------------------
> 
> (Updated Dec. 11, 2013, 9:32 a.m.)
> 
> 
> Review request for cloudstack and Girish Shilamkar.
> 
> 
> Bugs: CLOUDSTACK-5443
>     https://issues.apache.org/jira/browse/CLOUDSTACK-5443
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> There was an issue with logging under userTestClient.. test code. More information inside the bug. 
> Fixed that.
> 
> 
> Diffs
> -----
> 
>   tools/marvin/marvin/cloudstackConnection.py 48a2d21 
>   tools/marvin/marvin/cloudstackTestClient.py 8a41fad 
>   tools/marvin/marvin/deployDataCenter.py 0a175d9 
> 
> Diff: https://reviews.apache.org/r/16177/diff/
> 
> 
> Testing
> -------
> 
> DeployDC and sanity testing worked fine. Ran BVT against simulator
> 
> 
> Thanks,
> 
> Santhosh Edukulla
> 
>

Re: Review Request 16177: Fixed CLOUDSTACK-5443

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/16177/#review30196
-----------------------------------------------------------


Commit c05d94a51feeedfe91eeb32c4e0a0d5c83b7031b in branch refs/heads/4.3 from Santhosh Edukulla
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=c05d94a ]

CLOUDSTACK-5443: Fixed the issue.


- ASF Subversion and Git Services


On Dec. 11, 2013, 9:32 a.m., Santhosh Edukulla wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/16177/
> -----------------------------------------------------------
> 
> (Updated Dec. 11, 2013, 9:32 a.m.)
> 
> 
> Review request for cloudstack and Girish Shilamkar.
> 
> 
> Bugs: CLOUDSTACK-5443
>     https://issues.apache.org/jira/browse/CLOUDSTACK-5443
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> There was an issue with logging under userTestClient.. test code. More information inside the bug. 
> Fixed that.
> 
> 
> Diffs
> -----
> 
>   tools/marvin/marvin/cloudstackConnection.py 48a2d21 
>   tools/marvin/marvin/cloudstackTestClient.py 8a41fad 
>   tools/marvin/marvin/deployDataCenter.py 0a175d9 
> 
> Diff: https://reviews.apache.org/r/16177/diff/
> 
> 
> Testing
> -------
> 
> DeployDC and sanity testing worked fine. Ran BVT against simulator
> 
> 
> Thanks,
> 
> Santhosh Edukulla
> 
>

Re: Review Request 16177: Fixed CLOUDSTACK-5443

Posted by Girish Shilamkar <gi...@clogeny.com>.

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

Ship it!


Committed to 4.2, 4.3 and marvin.

- Girish Shilamkar


On Dec. 11, 2013, 9:32 a.m., Santhosh Edukulla wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/16177/
> -----------------------------------------------------------
> 
> (Updated Dec. 11, 2013, 9:32 a.m.)
> 
> 
> Review request for cloudstack and Girish Shilamkar.
> 
> 
> Bugs: CLOUDSTACK-5443
>     https://issues.apache.org/jira/browse/CLOUDSTACK-5443
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> There was an issue with logging under userTestClient.. test code. More information inside the bug. 
> Fixed that.
> 
> 
> Diffs
> -----
> 
>   tools/marvin/marvin/cloudstackConnection.py 48a2d21 
>   tools/marvin/marvin/cloudstackTestClient.py 8a41fad 
>   tools/marvin/marvin/deployDataCenter.py 0a175d9 
> 
> Diff: https://reviews.apache.org/r/16177/diff/
> 
> 
> Testing
> -------
> 
> DeployDC and sanity testing worked fine. Ran BVT against simulator
> 
> 
> Thanks,
> 
> Santhosh Edukulla
> 
>