Skip to content

Commit

Permalink
rollback to cache :(
Browse files Browse the repository at this point in the history
  • Loading branch information
miparnisari committed Oct 20, 2023
1 parent 0e60c02 commit 6e9870a
Show file tree
Hide file tree
Showing 2 changed files with 37 additions and 56 deletions.
28 changes: 27 additions & 1 deletion .github/workflows/main.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -18,4 +18,30 @@ jobs:
with:
go-version-file: './go.mod'
cache-dependency-path: './go.sum'
check-latest: true
check-latest: true

# Run benchmark with `go test -bench` and stores the output to a file
- name: Run benchmark
run: |
set -o pipefail
make bench | tee bench_output.txt
- name: Get JSON for benchmark
uses: benchmark-action/github-action-benchmark@70405016b032d44f409e4b1b451c40215cbe2393 # v1.18.0
with:
# What benchmark tool the output.txt came from
tool: 'go'
# Where the output from the benchmark tool is stored
output-file-path: bench_output.txt
# Updates the file content instead of generating a Git commit in GitHub Pages branch
external-data-json-path: ./cache/benchmark-data.json
# Workflow will fail when an alert happens
fail-on-alert: true
github-token: ${{ secrets.GITHUB_TOKEN }}
comment-on-alert: true

- name: Save benchmark data
uses: actions/cache/save@704facf57e6136b1bc63b828d79edcd491f0ee84 # v3.3.2
with:
path: ./cache/benchmark-data.json
key: ${{ runner.os }}-go-benchmark
65 changes: 10 additions & 55 deletions .github/workflows/prs.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,48 +4,8 @@ on:
branches:
- main

permissions:
contents: write
pull-requests: write

jobs:
bench-main:
runs-on: ubuntu-latest
timeout-minutes: 15
steps:
- name: Checkout main
uses: actions/checkout@8ade135a41bc03ea155e62e844d188df1ea18608 # v3.5.2
with:
ref: main

- uses: actions/setup-go@93397bea11091df50f3d7e59dc26a7711a8bcfbe # v4.1.0
with:
go-version-file: './go.mod'
cache-dependency-path: './go.sum'
check-latest: true

- name: Run benchmark of main
run: |
set -o pipefail
make bench | tee main_bench_output.txt
- name: Store benchmark result of main
uses: benchmark-action/github-action-benchmark@70405016b032d44f409e4b1b451c40215cbe2393 # v1.18.0
with:
# What benchmark tool the output.txt came from
tool: 'go'
# Where the output from the benchmark tool is stored
output-file-path: main_bench_output.txt
# Where the output from the benchmark will be stored
external-data-json-path: ./main_bench_output.json
save-data-file: true
fail-on-alert: false
- uses: actions/upload-artifact@v3
with:
name: benchmarks
path: ./main_bench_output.json

bench-PR:
go-bench:
runs-on: ubuntu-latest
timeout-minutes: 15
steps:
Expand All @@ -57,24 +17,19 @@ jobs:
cache-dependency-path: './go.sum'
check-latest: true

- name: Run benchmark of PR
# Run benchmark with `go test -bench` and stores the output to a file
- name: Run benchmark
run: |
set -o pipefail
make bench | tee ${{ github.sha }}_bench_output.txt
- uses: actions/upload-artifact@v3
with:
name: benchmarks
path: ${{ github.sha }}_bench_output.txt
compare-pr:
runs-on: ubuntu-latest
needs: [bench-main, bench-pr]
timeout-minutes: 15
steps:
- name: Downloads benchmarks
uses: actions/download-artifact@v3
# Download previous benchmark result from cache
- name: Download previous benchmark data
uses: actions/cache/restore@704facf57e6136b1bc63b828d79edcd491f0ee84 # v3.3.2
with:
name: benchmarks
fail-on-cache-miss: true
path: ./cache/benchmark-data.json
key: ${{ runner.os }}-go-benchmark

- name: Compare benchmark result
uses: benchmark-action/github-action-benchmark@70405016b032d44f409e4b1b451c40215cbe2393 # v1.18.0
Expand All @@ -84,7 +39,7 @@ jobs:
# Where the output from the benchmark tool is stored
output-file-path: ${{ github.sha }}_bench_output.txt
# Where the previous data file is stored
external-data-json-path: ./main_bench_output.json
external-data-json-path: ./cache/benchmark-data.json
# Do not save the data
save-data-file: false
# Workflow will fail when an alert happens
Expand Down

0 comments on commit 6e9870a

Please sign in to comment.