cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Santhosh Edukulla" <santhosh.eduku...@citrix.com>
Subject Review Request 21070: Fixed few exception cases.
Date Mon, 05 May 2014 06:49:36 GMT

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

Review request for cloudstack and sanjeev n.


Repository: cloudstack-git


Description
-------

Fixed few exception cases. When some request fails, it throws up the stack, but the message
str..does not contains a given attribute is little misleading. So, raising exceptions for
all cases.
Added testsuite name for userprovided\default log folder path
Fixed pep8 changes.


Diffs
-----

  tools/marvin/marvin/cloudstackConnection.py caa8609 
  tools/marvin/marvin/cloudstackTestClient.py d3a6b94 
  tools/marvin/marvin/configGenerator.py 4f03fd0 
  tools/marvin/marvin/marvinInit.py de580ce 
  tools/marvin/marvin/marvinLog.py 65687aa 
  tools/marvin/marvin/marvinPlugin.py c817cd6 

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


Testing
-------


Thanks,

Santhosh Edukulla


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message