Skip to content

Commit 000874b

Browse files
committed
Fix permissions on wstest directories before artifact upload
Docker containers may create files as root, making them read-only to the GitHub Actions runner user. The verified upload action needs write access to create CHECKSUMS.sha256 in the artifact directory. Add chmod -R u+w steps before both client and server report uploads. Note: This work was completed with AI assistance (Claude Code).
1 parent 85c94df commit 000874b

1 file changed

Lines changed: 16 additions & 0 deletions

File tree

.github/workflows/wstest.yml

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -203,6 +203,14 @@ jobs:
203203
echo "==> Without-NVX client reports:"
204204
ls -la .wstest/clients-without-nvx/ | head -10 || echo "Not found"
205205
206+
# Fix permissions on wstest directories (Docker may create files as root)
207+
# The verified upload action needs write access to create CHECKSUMS.sha256
208+
- name: Fix permissions on client reports
209+
if: always()
210+
run: |
211+
chmod -R u+w .wstest/clients-with-nvx/ 2>/dev/null || true
212+
chmod -R u+w .wstest/clients-without-nvx/ 2>/dev/null || true
213+
206214
- name: Upload client reports (with-nvx)
207215
uses: wamp-proto/wamp-cicd/actions/upload-artifact-verified@main
208216
if: always()
@@ -415,6 +423,14 @@ jobs:
415423
echo "✅ All server tests completed for BOTH NVX configurations"
416424
echo "======================================================================"
417425
426+
# Fix permissions on wstest directories (Docker may create files as root)
427+
# The verified upload action needs write access to create CHECKSUMS.sha256
428+
- name: Fix permissions on server reports
429+
if: always()
430+
run: |
431+
chmod -R u+w .wstest/servers-with-nvx/ 2>/dev/null || true
432+
chmod -R u+w .wstest/servers-without-nvx/ 2>/dev/null || true
433+
418434
- name: Upload server reports (with-nvx)
419435
uses: wamp-proto/wamp-cicd/actions/upload-artifact-verified@main
420436
if: always()

0 commit comments

Comments
 (0)