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

Proposed by Javier Collado
Status: Merged
Approved by: Javier Collado
Approved revision: 722
Merged at revision: 724
Proposed branch: lp:~javier.collado/utah/bug1068664-2
Merge into: lp:utah
Diff against target: 12 lines (+1/-1)
1 file modified
utah/run.py (+1/-1)
To merge this branch: bzr merge lp:~javier.collado/utah/bug1068664-2
Reviewer Review Type Date Requested Status
Joe Talbott (community) Approve
Review via email: mp+131387@code.launchpad.net

Description of the change

As suggested by Max, setting "root=True" in the call to the utah client,
fixes the problem. Much easier than my previous attempt to solve the problem.

To post a comment you must log in.
Revision history for this message
Joe Talbott (joetalbott) wrote :

Approve = True

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-10-16 10:46:10 +0000
+++ utah/run.py 2012-10-25 12:12:19 +0000
@@ -40,7 +40,7 @@
40 options = ' -r /tmp/' + os.path.basename(locallist)40 options = ' -r /tmp/' + os.path.basename(locallist)
41 utah_command = 'utah' + extraopts + options41 utah_command = 'utah' + extraopts + options
42 try:42 try:
43 _returncode, stdout, _stderr = machine.run(utah_command)43 _returncode, stdout, _stderr = machine.run(utah_command, root=True)
44 # TODO: Decide which returncode means utah client failure44 # TODO: Decide which returncode means utah client failure
45 # and which one means test case failure45 # and which one means test case failure
46 except UTAHException as error:46 except UTAHException as error:

Subscribers

People subscribed via source and target branches