----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20659/#review41591 -----------------------------------------------------------
Commit 4f1f182cba5579da2fc7ce1f02019a0afa00efeb in cloudstack's branch refs/heads/4.4-automation from SrikanteswaraRao Talluri [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=4f1f182 ] CLOUDSTACK-5674:Fixed pep8 errors in python files in marvin folder Signed-off-by: SrikanteswaraRao Talluri <tall...@apache.org> - ASF Subversion and Git Services On April 24, 2014, 9:15 a.m., Gaurav Aradhye wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/20659/ > ----------------------------------------------------------- > > (Updated April 24, 2014, 9:15 a.m.) > > > Review request for cloudstack and Santhosh Edukulla. > > > Bugs: CLOUDSTACK-5674 > https://issues.apache.org/jira/browse/CLOUDSTACK-5674 > > > Repository: cloudstack-git > > > Description > ------- > > In case of well handled exceptions returned from the server with proper error > message, the response was as it is returned. Instead, it should check the > jobresultcode. If the jobresultcode is 530, it means that exception is > returned from server. In this case we should return FAILED. > > > Diffs > ----- > > tools/marvin/marvin/cloudstackConnection.py d006002 > tools/marvin/marvin/codes.py 4d44c58 > > Diff: https://reviews.apache.org/r/20659/diff/ > > > Testing > ------- > > Yes > > > Thanks, > > Gaurav Aradhye > >