Don't publish benchmark results from non-main branches

Signed-off-by: James Bornholt <bornholt@amazon.com>
jamesbornholt · Mar 13, 2024 · e46cf4a · e46cf4a
1 parent b74b6b4
commit e46cf4a
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 5 deletions.
diff --git a/.github/workflows/bench.yml b/.github/workflows/bench.yml
@@ -7,6 +7,9 @@ on:
     inputs:
       environment:
         type: string
+      publish:
+        type: boolean
+        default: false
       ref:
         required: true
         type: string
@@ -62,7 +65,7 @@ jobs:
         # GitHub API token to make a commit comment
         github-token: ${{ secrets.GITHUB_TOKEN }}
         # Store the results and deploy GitHub pages automatically if the results are from main branch
-        auto-push: ${{ inputs.environment && 'false' || 'true' }}
+        auto-push: ${{ inputs.publish }}
         comment-on-alert: true
         max-items-in-chart: 20
 
@@ -108,7 +111,7 @@ jobs:
         # GitHub API token to make a commit comment
         github-token: ${{ secrets.GITHUB_TOKEN }}
         # Store the results and deploy GitHub pages automatically if the results are from main branch
-        auto-push: ${{ inputs.environment && 'false' || 'true' }}
+        auto-push: ${{ inputs.publish }}
         comment-on-alert: true
         max-items-in-chart: 20
 
@@ -156,6 +159,6 @@ jobs:
         # GitHub API token to make a commit comment
         github-token: ${{ secrets.GITHUB_TOKEN }}
         # Store the results and deploy GitHub pages automatically if the results are from main branch
-        auto-push: ${{ inputs.environment && 'false' || 'true' }}
+        auto-push: ${{ inputs.publish }}
         comment-on-alert: true
         max-items-in-chart: 20
diff --git a/.github/workflows/bench_main.yml b/.github/workflows/bench_main.yml
@@ -14,8 +14,10 @@ jobs:
     uses: ./.github/workflows/bench.yml
     with:
       ref: ${{ github.event.after }}
+      publish: ${{ github.event.ref == 'refs/heads/main' }}
   s3express-integration:
     name: Benchmarks (s3express)
     uses: ./.github/workflows/bench_s3express.yml
     with:
       ref: ${{ github.event.after }}
+      publish: ${{ github.event.ref == 'refs/heads/main' }}
diff --git a/.github/workflows/bench_s3express.yml b/.github/workflows/bench_s3express.yml
@@ -7,6 +7,9 @@ on:
     inputs:
       environment:
         type: string
+      publish:
+        type: boolean
+        default: false
       ref:
         required: true
         type: string
@@ -63,7 +66,7 @@ jobs:
         # GitHub API token to make a commit comment
         github-token: ${{ secrets.GITHUB_TOKEN }}
         # Store the results and deploy GitHub pages automatically if the results are from main branch
-        auto-push: ${{ inputs.environment && 'false' || 'true' }}
+        auto-push: ${{ inputs.publish }}
         comment-on-alert: true
         max-items-in-chart: 20
 
@@ -109,6 +112,6 @@ jobs:
         # GitHub API token to make a commit comment
         github-token: ${{ secrets.GITHUB_TOKEN }}
         # Store the results and deploy GitHub pages automatically if the results are from main branch
-        auto-push: ${{ inputs.environment && 'false' || 'true' }}
+        auto-push: ${{ inputs.publish }}
         comment-on-alert: true
         max-items-in-chart: 20