Merge ~andrey-fedoseev/launchpad-buildd:backend-open into launchpad-buildd:master
Proposed by
Andrey Fedoseev
| Status: | Merged |
|---|---|
| Approved by: | Andrey Fedoseev |
| Approved revision: | de88cd3e744e161e63dcceba84101d145dbc010c |
| Merge reported by: | Otto Co-Pilot |
| Merged at revision: | not available |
| Proposed branch: | ~andrey-fedoseev/launchpad-buildd:backend-open |
| Merge into: | launchpad-buildd:master |
| Diff against target: |
41 lines (+8/-2) 2 files modified
lpbuildd/target/chroot.py (+6/-2) lpbuildd/target/tests/test_chroot.py (+2/-0) |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Colin Watson (community) | Approve | ||
|
Review via email:
|
|||
Commit message
Fix the bug with `Backend.open()` caused by file ownership in chroot environment
Description of the change
Previously, `Chroot.copy_out()` produced a file owned by root which prevented `Backend.open()` to open it for writing.
Now, the file created by `Chroot.copy_out()` is owned by `buildd`
To post a comment you must log in.
Revision history for this message
| Colin Watson (cjwatson) | # |
review:
Approve
There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
