Merge branch 'main' into fake-scheduling-improvement

DerekFrank · Feb 28, 2025 · 5f1e3c9 · 5f1e3c9
2 parents e08359b + 0fd18f9
commit 5f1e3c9
Showing 1 changed file with 4 additions and 8 deletions.
diff --git a/.github/workflows/scheduling-benchmarking.yaml b/.github/workflows/scheduling-benchmarking.yaml
@@ -20,7 +20,7 @@ jobs:
           ref: ${{ github.event.pull_request.head.sha }}
           fetch-depth: 0
       - name: Run Test
-        id: test-run
+        id: test-run-before
         run: |
           {
             cd pkg/controllers/provisioning/scheduling
@@ -29,16 +29,14 @@ jobs:
             echo EOF
           } >> "$GITHUB_OUTPUT"
       - uses: actions/github-script@v7
-        env:
-          BEFORE_OUTPUT: ${{ steps.test-run.outputs.BEFORE_OUTPUT }}
         with:
           github-token: ${{secrets.GITHUB_TOKEN}}
           script: |
             github.rest.issues.createComment({
               issue_number: context.issue.number,
               owner: context.repo.owner,
               repo: context.repo.repo,
-              body: steps.test-run.outputs.BEFORE_OUTPUT
+              body: jobs.before.steps.test-run-before.outputs.BEFORE_OUTPUT
             })
   after:
     permissions:
@@ -50,7 +48,7 @@ jobs:
         with:
           fetch-depth: 0
       - name: Run Test
-        id: test-run
+        id: test-run-after
         run: |
           {
             cd pkg/controllers/provisioning/scheduling
@@ -59,14 +57,12 @@ jobs:
             echo EOF
           } >> "$GITHUB_OUTPUT"
       - uses: actions/github-script@v7
-        env:
-          BEFORE_OUTPUT: ${{ steps.test-run.outputs.AFTER_OUTPUT }}
         with:
           github-token: ${{secrets.GITHUB_TOKEN}}
           script: |
             github.rest.issues.createComment({
               issue_number: context.issue.number,
               owner: context.repo.owner,
               repo: context.repo.repo,
-              body: '${{ steps.test-run.outputs.AFTER_OUTPUT }}'
+              body: jobs.before.steps.test-run-after.outputs.AFTER_OUTPUT
             })