----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21070/#review42142 -----------------------------------------------------------
Please rebase with master since patch apply failed on master branch. - sanjeev n On May 5, 2014, 6:49 a.m., Santhosh Edukulla wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/21070/ > ----------------------------------------------------------- > > (Updated May 5, 2014, 6:49 a.m.) > > > 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 > >
