From a82fa8b35eba155b17d051b13d9ead9f611d68ba Mon Sep 17 00:00:00 2001
From: Pepijn <pepijn@huggingface.co>
Date: Wed, 8 Apr 2026 19:52:49 +0200
Subject: [PATCH] fix(ci): re-chmod artifacts after eval to fix unreadable
 files

Files created by user_lerobot inside the eval container inherit a
restrictive umask, making them unreadable by the runner after the
container exits. Add a post-eval 'docker run --user root' chmod step
so upload-artifact can find the video files.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .github/workflows/benchmark_tests.yml | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git a/.github/workflows/benchmark_tests.yml b/.github/workflows/benchmark_tests.yml
index b2a37a9b3..8214ca735 100644
--- a/.github/workflows/benchmark_tests.yml
+++ b/.github/workflows/benchmark_tests.yml
@@ -133,6 +133,16 @@ jobs:
                 --output_dir=/artifacts
             "
 
+      - name: Fix Libero artifact permissions
+        if: always()
+        run: |
+          # New files written by user_lerobot inside the container inherit a
+          # restrictive umask; re-chmod as root so the runner can read them.
+          docker run --rm --user root \
+            -v /tmp/libero-artifacts:/artifacts \
+            lerobot-benchmark-libero:ci \
+            bash -c "chmod -R 777 /artifacts"
+
       - name: Parse Libero eval metrics
         if: always()
         run: |
@@ -222,6 +232,14 @@ jobs:
                 --output_dir=/artifacts
             "
 
+      - name: Fix MetaWorld artifact permissions
+        if: always()
+        run: |
+          docker run --rm --user root \
+            -v /tmp/metaworld-artifacts:/artifacts \
+            lerobot-benchmark-metaworld:ci \
+            bash -c "chmod -R 777 /artifacts"
+
       - name: Parse MetaWorld eval metrics
         if: always()
         run: |