Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(162)

Issue 1871113002: Stop choosing $TMPDIR as a safe place to create hardlinks on OSX. (Closed)

Created:
4 years, 8 months ago by M-A Ruel
Modified:
4 years, 8 months ago
CC:
chromium-reviews, infra-reviews+luci-py_chromium.org, Michael Achenbach
Base URL:
git@github.com:luci/luci-py.git@master
Target Ref:
refs/heads/master
Project:
luci-py
Visibility:
Public.

Description

Stop choosing $TMPDIR as a safe place to create hardlinks on OSX. - Hardlinking doesn't update the inode modification time. - /System/Library/LaunchDaemons/com.apple.bsd.dirhelper.plist runs every day at 3:35am and deletes any file in $TMPDIR that is older than 3 days old. - This means any isolated task mapped in $TMPDIR using files in the cache older than 3 days will get deleted my dirhelper if the task is running at 3:35am. - Frustration while trying to diagnose this race condition ensue. R=vadimsh@chromium.org BUG= Committed: https://github.com/luci/luci-py/commit/90eecffb024b23387686702006cc4f1fa3f957a5

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -3 lines) Patch
M client/run_isolated.py View 1 chunk +12 lines, -3 lines 0 comments Download

Messages

Total messages: 14 (8 generated)
M-A Ruel
4 years, 8 months ago (2016-04-08 19:03:19 UTC) #1
Vadim Sh.
oh lgtm
4 years, 8 months ago (2016-04-08 19:11:53 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1871113002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1871113002/1
4 years, 8 months ago (2016-04-08 20:35:03 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1871113002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1871113002/1
4 years, 8 months ago (2016-04-08 20:38:13 UTC) #10
commit-bot: I haz the power
Committed patchset #1 (id:1) as https://github.com/luci/luci-py/commit/90eecffb024b23387686702006cc4f1fa3f957a5
4 years, 8 months ago (2016-04-08 20:40:00 UTC) #12
Michael Achenbach
4 years, 8 months ago (2016-04-11 07:03:06 UTC) #14
Message was sent while issue was closed.
Thanks for the fix! Lets hope that was the root cause...

Powered by Google App Engine
This is Rietveld 408576698