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/08 11:49:20 UTC

Review Request 21197: Added port issue and user apiclient fix issue

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

Review request for cloudstack and Koushik Das.


Repository: cloudstack-git


Description
-------

Fixed a refernece issue with mgmt_details variable.
Fixed user api client creation issue for using proper port


Diffs
-----

  tools/marvin/marvin/cloudstackTestClient.py 9fe559d 

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


Testing
-------

Tested the changes with simulator run


Thanks,

Santhosh Edukulla

Re: Review Request 21197: Added port issue and user apiclient fix issue

Posted by Koushik Das <ko...@citrix.com>.

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

Ship it!


4.4-forward -> 7734ece8f1455c6ad39546b4f204257223036716
master -> e69a94b61ee160532d660165e8b715f0f3f83ea9

- Koushik Das


On May 8, 2014, 9:49 a.m., Santhosh Edukulla wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/21197/
> -----------------------------------------------------------
> 
> (Updated May 8, 2014, 9:49 a.m.)
> 
> 
> Review request for cloudstack and Koushik Das.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> Fixed a refernece issue with mgmt_details variable.
> Fixed user api client creation issue for using proper port
> 
> 
> Diffs
> -----
> 
>   tools/marvin/marvin/cloudstackTestClient.py 9fe559d 
> 
> Diff: https://reviews.apache.org/r/21197/diff/
> 
> 
> Testing
> -------
> 
> Tested the changes with simulator run
> 
> 
> Thanks,
> 
> Santhosh Edukulla
> 
>