----------------------------------------------------------- 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
