chore: Fix benchmark file path, run benchmark on PRs (#14220)

Run benchmark on PR, but skip publishing and notifications
prisma · Jul 8, 2022 · 135d769 · 135d769
1 parent 8d99655
commit 135d769
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 4 deletions.
diff --git a/.github/workflows/benchmark.yml b/.github/workflows/benchmark.yml
@@ -1,5 +1,4 @@
 name: Benchmark
-# Run on push only for main, if not it will trigger push & pull_request on PRs at the same time
 on:
   push:
     branches:
@@ -20,6 +19,19 @@ on:
       - '.dockerignore'
       - 'scripts/ci/publish.ts'
       - 'graphs/**'
+  pull_request:
+    paths-ignore:
+      - '*.md'
+      - '.devcontainer/**'
+      - '.vscode/**'
+      - 'docs/**'
+      - 'examples/**'
+      - 'LICENSE'
+      - '.github/CODEOWNERS'
+      - 'src/.buildkite/**'
+      - '.dockerignore'
+      - 'scripts/ci/publish.ts'
+      - 'graphs/**'
 
 env:
   PRISMA_TELEMETRY_INFORMATION: 'prisma benchmark.yml'
@@ -53,6 +65,7 @@ jobs:
         run: pnpm run bench
 
       - name: Store benchmark result
+        if: ${{ github.ref == 'refs/heads/main' }}
         uses: rhysd/github-action-benchmark@v1
         with:
           name: Benchmark.js Benchmark
@@ -68,11 +81,11 @@ jobs:
           alert-comment-cc-users: '@Jolg42,@millsp,@aqrln,@SevInf,@danstarns,@jkomyno'
 
       - name: 'Set current job url in SLACK_FOOTER env var'
-        if: ${{ failure() }}
+        if: ${{ failure() && github.ref == 'refs/heads/main' }}
         run: echo "SLACK_FOOTER=<$GITHUB_SERVER_URL/$GITHUB_REPOSITORY/actions/runs/$GITHUB_RUN_ID|Click here to go to the job logs>" >> $GITHUB_ENV
 
       - name: Slack Notification on Failure
-        if: ${{ failure() }}
+        if: ${{ failure() && github.ref == 'refs/heads/main' }}
         uses: rtCamp/action-slack-notify@v2.2.0
         env:
           SLACK_TITLE: 'Benchmark failed :x:'

diff --git a/packages/client/src/__tests__/benchmarks/huge-schema/huge-schema.bench.ts b/packages/client/src/__tests__/benchmarks/huge-schema/huge-schema.bench.ts
@@ -48,7 +48,7 @@ suite
     printSize('./node_modules/.prisma/client/index.js')
     // For GitHub CI
     if (process.env.CI) {
-      printSize('./node_modules/.prisma/client/query-engine-debian-openssl-1.1.x')
+      printSize('./node_modules/.prisma/client/libquery_engine-debian-openssl-1.1.x.so.node')
     }
     execa.sync('rm', ['-rf', `./dotPlusAtPrismaClientFolder.zip`], {
       stdout: 'pipe',