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
1=== modified file 'utah/run.py'
2--- utah/run.py 2012-12-13 18:48:32 +0000
3+++ utah/run.py 2013-01-04 09:47:36 +0000
4@@ -103,10 +103,6 @@
5 traceback.print_exc()
6 machine.logger.error('Failed to install client. '
7 'Writing empty reports.')
8-
9- # Make sure that default logs are written
10- for runlist in args.runlists:
11- _write(runlist)
12 exitstatus = 1
13 else:
14 # Server will return success code only if the execution

Subscribers

People subscribed via source and target branches