Merge lp://staging/~nuclearbob/utah/bug1073617 into lp://staging/utah
Proposed by
Max Brustkern
Status: | Work in progress |
---|---|
Proposed branch: | lp://staging/~nuclearbob/utah/bug1073617 |
Merge into: | lp://staging/utah |
Diff against target: |
48 lines (+18/-4) 2 files modified
utah/client/common.py (+4/-4) utah/client/tests/test_common.py (+14/-0) |
To merge this branch: | bzr merge lp://staging/~nuclearbob/utah/bug1073617 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Max Brustkern (community) | Needs Fixing | ||
Javier Collado (community) | Needs Fixing | ||
Joe Talbott (community) | Needs Information | ||
Review via email: mp+135239@code.staging.launchpad.net |
Description of the change
This branch adds -i to the sudo command used to execute commands using run_as. This preserves the environment of the intended user.
A test for this environment preservation is also added, and an existing test that was broken by this change is updated to work.
To post a comment you must log in.
Unmerged revisions
- 759. By Max Brustkern
-
Fixed formatting per Javier's suggestion
- 758. By Max Brustkern
-
Changed manner of privilege switching so all tests pass, including the old test reverted back to the nobody user
- 757. By Max Brustkern
-
Merged latest changes from dev branch
- 756. By Max Brustkern
-
Added test for run_as environment and updated existing test to work with environment setting
- 755. By Max Brustkern
-
Added -i option to sudo to get user's environment
This is intended to fix this: /bugs.launchpad .net/utah/ +bug/1073617
https:/