restructuring benchmarks workflow, triggered only with merge on maste…

…r branch

.github/workflows/benchmark_template.yml → .github/workflows/benchmark-master-push.yml

@@ -1,18 +1,11 @@
-name: benchmarks
+name: run benchmarks when pushing on main branch
+
 on:
- workflow_call:
- inputs:
- publish:
- required: true
- type: boolean
- test-flags:
- required: true
- type: string
- runner-additional-tag:
- type: string
- default: benchmarks
- external-data-json-path:
- type: string
+ push:
+ branches:
+ - master
+
+ # publish: true
 
 permissions:
  # deployments permission to deploy GitHub pages website
@@ -26,7 +19,7 @@ env:
 jobs:
  benchmarks:
  if: ${{ github.repository == 'gnolang/gno' }}
- runs-on: [self-hosted, Linux, X64, "${{ inputs.runner-additional-tag }}"]
+ runs-on: [self-hosted, Linux, X64, benchmarks]
  steps:
  - name: Checkout
  uses: actions/checkout@v4
@@ -40,7 +33,7 @@ jobs:
  - name: Run benchmark
  run: |
  go test -benchmem -bench=. ./... -run=^$ \
- -cpu 1,2 ${{ inputs.test-flags }} | tee benchmarks.txt
+ -cpu 1,2 -timeout 50m | tee benchmarks.txt
 
  - name: Download previous benchmark data
  uses: actions/cache@v4
@@ -56,17 +49,18 @@ jobs:
  tool: 'go'
  output-file-path: benchmarks.txt
  # Where the previous data file is stored
- external-data-json-path: ${{ inputs.external-data-json-path }}
+ external-data-json-path: ./cache/benchmark-data.json
+ max-items-in-chart: 100
  # Show alert with commit comment on detecting possible performance regression
- alert-threshold: '20%'
+ alert-threshold: '100%'
  fail-on-alert: true
  comment-on-alert: true
+ alert-comment-cc-users: '@ajnavarro,@thehowl,@zivkovicmilos'
  # Enable Job Summary for PRs
  summary-always: true
  github-token: ${{ secrets.GITHUB_TOKEN }}
  # NOTE you need to use a separate GITHUB PAT token that has a write access to the specified repository.
  # gh-repository: 'github.com/gnolang/benchmarks' # on gh-pages branch
  gh-pages-branch: gh-benchmarks
  benchmark-data-dir-path: .
- auto-push: ${{ inputs.publish }}
- alert-comment-cc-users: '@ajnavarro,@thehowl,@zivkovicmilos'
+ auto-push: true