Skip to content

Conversation

jcristau
Copy link
Contributor

We don't want to leave a root-owned cache in the worker user's HOME directory.

We don't want to leave a root-owned cache in the worker user's HOME
directory.
@jcristau
Copy link
Contributor Author

Verified that the generated decision image doesn't have root-owned stuff in /builds/docker anymore:

root@53686a944a52:/# ls -la /builds/worker/
total 24
drwxr-xr-x 3 worker worker 4096 Oct  8 14:22 .
drwxr-xr-x 3 root   root   4096 Oct  8 14:22 ..
-rw-r--r-- 1 worker worker  220 Mar 29  2024 .bash_logout
-rw-r--r-- 1 worker worker 3526 Mar 29  2024 .bashrc
-rw-r--r-- 1 worker worker  807 Mar 29  2024 .profile
drwxr-xr-x 2 worker worker 4096 Oct  8 14:22 artifacts

jcristau added a commit to jcristau/reference-browser that referenced this pull request Oct 16, 2024
@jcristau
Copy link
Contributor Author

Bah, this is not enough, see https://firefox-ci-tc.services.mozilla.com/tasks/PiXh8GIQR1-NJSIGeJw9xQ/runs/0/logs/public/logs/live_backing.log

[pip-install 2024-10-16T11:13:03.385Z] executing ['uv', 'pip', 'install', '--python', '/usr/bin/python3', '--break-system-packages', '--require-hashes', '-r', '/builds/worker/checkouts/vcs/taskcluster/requirements.txt']
[pip-install 2024-10-16T11:13:03.437Z] error: failed to create file `/tmp/uv-b17927aca3d28934.lock`
[pip-install 2024-10-16T11:13:03.438Z]   Caused by: Permission denied (os error 13)

@ahal
Copy link
Collaborator

ahal commented Oct 16, 2024

That looks like a separate issue, so I think we can still merge this PR

Copy link
Collaborator

@ahal ahal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, I didn't think export would create a cache dir :/

@ahal
Copy link
Collaborator

ahal commented Oct 16, 2024

The codecov failure is a 503 on their end.

@ahal ahal merged commit 9e9b00b into taskcluster:main Oct 16, 2024
13 of 14 checks passed
@jcristau jcristau deleted the uv-no-cache branch October 17, 2024 08:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants