Ubuntu Test Cases

Merge lp://qastaging/~pwlars/ubuntu-test-cases/utah-sudo-fix-2 into lp://qastaging/ubuntu-test-cases/touch

utah-sudo-fix-2
Merge into touch

Proposed by Paul Larson on 2014-10-03

Status:	Merged
Merged at revision:	313
Proposed branch:	lp://qastaging/~pwlars/ubuntu-test-cases/utah-sudo-fix-2
Merge into:	lp://qastaging/ubuntu-test-cases/touch
Diff against target:	67 lines (+40/-15) 1 file modified scripts/jenkins.sh (+40/-15)
To merge this branch:	bzr merge lp://qastaging/~pwlars/ubuntu-test-cases/utah-sudo-fix-2
Related bugs:	Link a bug report

Reviewer	Review Type	Date Requested	Status
Evan (community)		2014-10-03	Approve on 2014-10-06
Review via email: mp+237141@code.qastaging.launchpad.net

Description of the change

This is just so that we can kill the utah dependency on sudo. After this, I think the only one we have left is pbuilder. Kind of brute force, but it works. Once we get all this converted to adt, we can eliminate this script anyway.

Revision history for this message

Evan (ev) wrote on 2014-10-06:

Some complaints, but otherwise +1.

review: Approve

Revision history for this message

Evan (ev) on 2014-10-06:

Revision history for this message

Paul Larson (pwlars) wrote on 2014-10-06:

I'll fix up the quoting and use $USER instead. Thanks for the comments. It turns out that the previously used approach *does* work properly (there's not a bug there as you fear where it chowns everything to root) but I do agree that your method is way better.