Merge branch 'main' into update_AMD_dsr1fp8_images

Author: cquil11

.github/workflows/label-validation.yml

@@ -7,7 +7,7 @@ concurrency:
 
 on:
   pull_request_target:
-    types: [labeled, synchronize]
+    types: labeled
 
 jobs:
   get-jobs:
@@ -83,12 +83,15 @@ jobs:
 
   validate:
     needs: get-jobs
-    if: ${{ needs.get-jobs.outputs.search-space-config != '[]' }}
+    if: ${{ always() && needs.get-jobs.result == 'success' && needs.get-jobs.outputs.search-space-config != '[]' }}
     uses: ./.github/workflows/benchmark-tmpl.yml
     strategy:
       fail-fast: false
       matrix:
-        config: ${{ fromJson(needs.get-jobs.outputs.search-space-config) }}
+        # The '|| '[]'' fallback is required because GitHub evaluates the matrix expression
+        # before the 'if' condition. Without it, fromJson() throws a parse error when the
+        # output is empty/undefined (e.g., when get-jobs is pending approval or was skipped).
+        config: ${{ fromJson(needs.get-jobs.outputs.search-space-config || '[]') }}
     secrets: inherit
     name: validate ${{ matrix.config.runner }}
     with:
@@ -107,14 +110,14 @@ jobs:
       conc: ${{ matrix.config.conc }}
 
   collect-results:
-    needs: validate
-    if: ${{ always() }}
+    needs: [get-jobs, validate]
+    if: ${{ always() && needs.get-jobs.result == 'success' }}
     uses: ./.github/workflows/collect-results.yml
     secrets: inherit
 
   calc-success-rate:
-    needs: collect-results
-    if: ${{ always() }}
+    needs: [get-jobs, collect-results]
+    if: ${{ always() && needs.get-jobs.result == 'success' }}
     runs-on: ubuntu-latest
 
     env:

benchmarks/benchmark_lib.sh

@@ -57,7 +57,7 @@ wait_for_server_ready() {
     fi
 
     # Show logs until server is ready
-    tail -f "$server_log" &
+    tail -f -n +1 "$server_log" &
     local TAIL_PID=$!
     until curl --output /dev/null --silent --fail http://0.0.0.0:$port/health; do
         if ! kill -0 "$server_pid" 2>/dev/null; then