Skip to content

Commit 79a5aa8

Browse files
AlexMikhalevclaude
andcommitted
fix(ci): add chown before chmod for self-hosted runner target cleanup
The checkout action fails with EACCES on root-owned files left by Docker builds. chmod alone makes files writable but doesn't change ownership, so the checkout cleanup still can't unlink them. Adding chown to transfer ownership back to the runner user before chmod. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent be1125c commit 79a5aa8

File tree

1 file changed

+7
-0
lines changed

1 file changed

+7
-0
lines changed

.github/workflows/ci-main.yml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,7 @@ jobs:
4444
- name: Fix target directory permissions
4545
run: |
4646
if [ -d "target" ]; then
47+
sudo chown -R $(id -u):$(id -g) target 2>/dev/null || true
4748
sudo chmod -R u+rw target 2>/dev/null || chmod -R u+rw target 2>/dev/null || true
4849
fi
4950
@@ -103,6 +104,7 @@ jobs:
103104
- name: Fix target directory permissions
104105
run: |
105106
if [ -d "target" ]; then
107+
sudo chown -R $(id -u):$(id -g) target 2>/dev/null || true
106108
sudo chmod -R u+rw target 2>/dev/null || chmod -R u+rw target 2>/dev/null || true
107109
fi
108110
@@ -208,6 +210,7 @@ jobs:
208210
- name: Fix target directory permissions
209211
run: |
210212
if [ -d "target" ]; then
213+
sudo chown -R $(id -u):$(id -g) target 2>/dev/null || true
211214
sudo chmod -R u+rw target 2>/dev/null || chmod -R u+rw target 2>/dev/null || true
212215
fi
213216
@@ -261,6 +264,7 @@ jobs:
261264
- name: Fix target directory permissions
262265
run: |
263266
if [ -d "target" ]; then
267+
sudo chown -R $(id -u):$(id -g) target 2>/dev/null || true
264268
sudo chmod -R u+rw target 2>/dev/null || chmod -R u+rw target 2>/dev/null || true
265269
fi
266270
@@ -308,6 +312,7 @@ jobs:
308312
- name: Fix target directory permissions
309313
run: |
310314
if [ -d "target" ]; then
315+
sudo chown -R $(id -u):$(id -g) target 2>/dev/null || true
311316
sudo chmod -R u+rw target 2>/dev/null || chmod -R u+rw target 2>/dev/null || true
312317
fi
313318
@@ -373,6 +378,7 @@ jobs:
373378
- name: Fix target directory permissions
374379
run: |
375380
if [ -d "target" ]; then
381+
sudo chown -R $(id -u):$(id -g) target 2>/dev/null || true
376382
sudo chmod -R u+rw target 2>/dev/null || chmod -R u+rw target 2>/dev/null || true
377383
fi
378384
@@ -430,6 +436,7 @@ jobs:
430436
- name: Fix target directory permissions
431437
run: |
432438
if [ -d "target" ]; then
439+
sudo chown -R $(id -u):$(id -g) target 2>/dev/null || true
433440
sudo chmod -R u+rw target 2>/dev/null || chmod -R u+rw target 2>/dev/null || true
434441
fi
435442

0 commit comments

Comments
 (0)