fix

Signed-off-by: alec-flowers <[email protected]>

Author: alec-flowers

.github/workflows/container-validation-backends.yml

@@ -30,6 +30,9 @@ jobs:
 
   backend-status-check:
     runs-on: ubuntu-latest
+    permissions:
+      actions: read
+      statuses: write
     steps:
       - name: "Check if builds will run and post status"
         uses: actions/github-script@v7
@@ -39,38 +42,44 @@ jobs:
             const owner = context.repo.owner;
             const repo = context.repo.repo;
 
-            // Wait a bit for jobs to start
-            await new Promise(resolve => setTimeout(resolve, 5000));
+            // Poll for jobs to appear, waiting up to 30 seconds
+            let backendJobs = [];
+            const backendJobNames = ['vllm', 'sglang', 'trtllm', 'operator'];
 
-            // Get the current workflow run
-            const workflowRun = await github.rest.actions.getWorkflowRun({
-              owner,
-              repo,
-              run_id: context.runId
-            });
+            for (let i = 0; i < 6; i++) {
+              await new Promise(resolve => setTimeout(resolve, 5000));
 
-            // List all jobs in this workflow run
-            const jobs = await github.rest.actions.listJobsForWorkflowRun({
-              owner,
-              repo,
-              run_id: context.runId
-            });
+              const jobs = await github.rest.actions.listJobsForWorkflowRun({
+                owner,
+                repo,
+                run_id: context.runId
+              });
 
-            // Check if any of the backend build jobs are running or queued
-            const backendJobNames = ['vllm', 'sglang', 'trtllm', 'operator'];
-            const backendJobs = jobs.data.jobs.filter(job =>
-              backendJobNames.some(name => job.name.includes(name))
-            );
+              backendJobs = jobs.data.jobs.filter(job =>
+                backendJobNames.some(name => job.name.includes(name))
+              );
+
+              console.log(`Poll ${i+1}: Found ${backendJobs.length} backend jobs`);
+
+              // If we found backend jobs, check their status
+              if (backendJobs.length > 0) {
+                break;
+              }
+            }
 
             const anyRunning = backendJobs.some(job =>
               job.status === 'queued' || job.status === 'in_progress'
             );
+            const anySkipped = backendJobs.some(job =>
+              job.status === 'completed' && job.conclusion === 'skipped'
+            );
 
             console.log(`Backend jobs found: ${backendJobs.length}`);
             console.log(`Any running: ${anyRunning}`);
+            console.log(`Any skipped: ${anySkipped}`);
 
-            if (!anyRunning && backendJobs.length > 0) {
-              // Builds are not running, post helpful message
+            if (backendJobs.length === 0 || (!anyRunning && backendJobs.length > 0)) {
+              // Builds are not running or all skipped, post helpful message
               const message = `Use /ok to test ${sha.substring(0, 7)} to run CI`;
 
               await github.rest.repos.createCommitStatus({
@@ -99,6 +108,8 @@ jobs:
     runs-on: ubuntu-latest
     needs: [vllm, sglang, trtllm, operator]
     if: always()
+    permissions:
+      statuses: write
     steps:
       - name: "Check all dependent jobs"
         uses: actions/github-script@v7