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/04/29 15:08:41 UTC

Review Request 20837: Fixed pep8 issues on master

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

Review request for cloudstack and SrikanteswaraRao Talluri.


Repository: cloudstack-git


Description
-------

Fixed pending pep8 issues, available due to merge changes


Diffs
-----

  tools/marvin/marvin/cloudstackConnection.py d006002 
  tools/marvin/marvin/codegenerator.py 1e02ddf 
  tools/marvin/marvin/codes.py 28e907c 
  tools/marvin/marvin/configGenerator.py 68ec24e 
  tools/marvin/marvin/deployAndRun.py d3b6b86 
  tools/marvin/marvin/jsonHelper.py 3f48a00 
  tools/marvin/marvin/marvinInit.py c35fec6 
  tools/marvin/marvin/marvinPlugin.py 4200a65 
  tools/marvin/marvin/sshClient.py 611c8b5 

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


Testing
-------


Thanks,

Santhosh Edukulla

Re: Review Request 20837: Fixed pep8 issues on master

Posted by SrikanteswaraRao Talluri <sr...@citrix.com>.

> On April 29, 2014, 2:06 p.m., SrikanteswaraRao Talluri wrote:
> > Ship It!

committed to master 56a44151aa31a8aac05b7a35d629af5714318ccb


- SrikanteswaraRao


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


On April 29, 2014, 1:08 p.m., Santhosh Edukulla wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20837/
> -----------------------------------------------------------
> 
> (Updated April 29, 2014, 1:08 p.m.)
> 
> 
> Review request for cloudstack and SrikanteswaraRao Talluri.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> Fixed pending pep8 issues, available due to merge changes
> 
> 
> Diffs
> -----
> 
>   tools/marvin/marvin/cloudstackConnection.py d006002 
>   tools/marvin/marvin/codegenerator.py 1e02ddf 
>   tools/marvin/marvin/codes.py 28e907c 
>   tools/marvin/marvin/configGenerator.py 68ec24e 
>   tools/marvin/marvin/deployAndRun.py d3b6b86 
>   tools/marvin/marvin/jsonHelper.py 3f48a00 
>   tools/marvin/marvin/marvinInit.py c35fec6 
>   tools/marvin/marvin/marvinPlugin.py 4200a65 
>   tools/marvin/marvin/sshClient.py 611c8b5 
> 
> Diff: https://reviews.apache.org/r/20837/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Santhosh Edukulla
> 
>

Re: Review Request 20837: Fixed pep8 issues on master

Posted by SrikanteswaraRao Talluri <sr...@citrix.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20837/#review41712
-----------------------------------------------------------

Ship it!


Ship It!

- SrikanteswaraRao Talluri


On April 29, 2014, 1:08 p.m., Santhosh Edukulla wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20837/
> -----------------------------------------------------------
> 
> (Updated April 29, 2014, 1:08 p.m.)
> 
> 
> Review request for cloudstack and SrikanteswaraRao Talluri.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> Fixed pending pep8 issues, available due to merge changes
> 
> 
> Diffs
> -----
> 
>   tools/marvin/marvin/cloudstackConnection.py d006002 
>   tools/marvin/marvin/codegenerator.py 1e02ddf 
>   tools/marvin/marvin/codes.py 28e907c 
>   tools/marvin/marvin/configGenerator.py 68ec24e 
>   tools/marvin/marvin/deployAndRun.py d3b6b86 
>   tools/marvin/marvin/jsonHelper.py 3f48a00 
>   tools/marvin/marvin/marvinInit.py c35fec6 
>   tools/marvin/marvin/marvinPlugin.py 4200a65 
>   tools/marvin/marvin/sshClient.py 611c8b5 
> 
> Diff: https://reviews.apache.org/r/20837/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Santhosh Edukulla
> 
>