Merge lp:~javier.collado/utah/bug1090013-2 into lp:utah

Proposed by Javier Collado
Status: Merged
Approved by: Max Brustkern
Approved revision: 794
Merged at revision: 794
Proposed branch: lp:~javier.collado/utah/bug1090013-2
Merge into: lp:utah
Diff against target: 14 lines (+0/-4)
1 file modified
utah/run.py (+0/-4)
To merge this branch: bzr merge lp:~javier.collado/utah/bug1090013-2
Reviewer Review Type Date Requested Status
Max Brustkern (community) Approve
Review via email: mp+141875@code.launchpad.net

Description of the change

An attempt to generate empty logs is still made when the client installation
fails. This branch removes the lines that are no longer needed.

To post a comment you must log in.
Revision history for this message
Max Brustkern (nuclearbob) wrote :

Looks good to me.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'utah/run.py'
--- utah/run.py 2012-12-13 18:48:32 +0000
+++ utah/run.py 2013-01-04 09:47:36 +0000
@@ -103,10 +103,6 @@
103 traceback.print_exc()103 traceback.print_exc()
104 machine.logger.error('Failed to install client. '104 machine.logger.error('Failed to install client. '
105 'Writing empty reports.')105 'Writing empty reports.')
106
107 # Make sure that default logs are written
108 for runlist in args.runlists:
109 _write(runlist)
110 exitstatus = 1106 exitstatus = 1
111 else:107 else:
112 # Server will return success code only if the execution108 # Server will return success code only if the execution

Subscribers

People subscribed via source and target branches