Closed Bug 1302821 Opened 3 years ago Closed 3 years ago
OSError: [Errno 13] Permission denied: '/home/worker/workspace/logs/log
58 bytes, text/x-review-board-request
We're seeing the error in the summary line in random tasks in automation. e.g. https://public-artifacts.taskcluster.net/Jwt-gb8wTmGhnbrxApZBUw/0/public/logs/live_backing.log My guess is this is fallout from switching tests to run-task, which is much more proactive and consistent about environment normalization and permissions adjustments.
Comment on attachment 8791339 [details] Bug 1302821 - Ensure owner has full privileges on directories; https://reviewboard.mozilla.org/r/78774/#review77360 This will fix the case where the directory contains subdirectories that are not writeable by the `worker` user. If that's the underlying issue, then this is the right fix. Otherwise, this doesn't hurt! ::: testing/docker/recipes/run-task:213 (Diff revision 1) > + if st.st_uid != uid or st.st_gid != gid: > + os.chown(full, uid, gid) > + > + # Also make sure dirs are writable in case we need to delete > + # them. > + if not st.st_mode & wanted_dir_mode: I think you want `st.st_mode & wanted_dir_mode == wanted_dir_mode` (that is, all wanted bits are set)
Attachment #8791339 - Flags: review?(dustin) → review+
Comment on attachment 8791339 [details] Bug 1302821 - Ensure owner has full privileges on directories; https://reviewboard.mozilla.org/r/78774/#review77362
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/4f48a94c9d5a Ensure owner has full privileges on directories; r=dustin
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
You need to log in before you can comment on or make changes to this bug.