Merge lp:~chris.macnaughton/mojo/error-output into lp:~ost-maintainers/mojo/py3

Proposed by Chris MacNaughton
Status: Merged
Merged at revision: 473
Proposed branch: lp:~chris.macnaughton/mojo/error-output
Merge into: lp:~ost-maintainers/mojo/py3
Diff against target: 19 lines (+1/-6)
1 file modified
mojo/phase.py (+1/-6)
To merge this branch: bzr merge lp:~chris.macnaughton/mojo/error-output
Reviewer Review Type Date Requested Status
Frode Nordahl Approve
Review via email: mp+350832@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Frode Nordahl (fnordahl) wrote :

LGTM, thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'mojo/phase.py'
--- mojo/phase.py 2018-07-17 11:57:20 +0000
+++ mojo/phase.py 2018-07-25 12:38:54 +0000
@@ -988,14 +988,9 @@
988 dl = mojo.juju.DebugLogs(988 dl = mojo.juju.DebugLogs(
989 workspace, stage, self.options.get('debug-logs'), self.options.get('debug-logs-stages-to-exclude'))989 workspace, stage, self.options.get('debug-logs'), self.options.get('debug-logs-stages-to-exclude'))
990 if dl.has_config:990 if dl.has_config:
991 output = e.output991 exception_output = "{}\n{}".format(e.output, dl.get_log_output())
992 if six.PY3:
993 output = output.decode('utf-8')
994 exception_output = "{}\n{}".format(output, dl.get_log_output())
995 else:992 else:
996 exception_output = e.output993 exception_output = e.output
997 if six.PY3:
998 exception_output = exception_output.decode('utf-8')
999 raise VerifyPhaseException(exception_output)994 raise VerifyPhaseException(exception_output)
1000 else:995 else:
1001 # Hooray, success!996 # Hooray, success!

Subscribers

People subscribed via source and target branches