Skip to content

Commit

Permalink
move bench_ec2_all to bench workflow and only store results for pushi…
Browse files Browse the repository at this point in the history
…ng to main event

Signed-off-by: Thing-han, Lim <[email protected]>
  • Loading branch information
potsrevennil committed Sep 19, 2024
1 parent 2ede9d4 commit b546dea
Show file tree
Hide file tree
Showing 2 changed files with 36 additions and 46 deletions.
36 changes: 36 additions & 0 deletions .github/workflows/bench.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@

name: Bench
on:
workflow_dispatch:
push:
branches: ["main"]
pull_request:
Expand Down Expand Up @@ -45,3 +46,38 @@ jobs:
store_results: ${{ github.repository_owner == 'pq-code-package' && github.ref == 'refs/heads/main' }}
bench_extra_args: ${{ matrix.target.bench_extra_args }}
gh_token: ${{ secrets.AWS_GITHUB_TOKEN }}
ec2_all:
name: ${{ matrix.target.name }} ${{ matrix.opt.name }}
permissions:
contents: 'read'
id-token: 'write'
strategy:
fail-fast: false
matrix:
opt:
- name: opt
value: true
- name: non-opt
value: false
target:
- name: Graviton2
ec2_instance_type: t4g.small
ec2_ami: ubuntu-latest (aarch64)
archflags: -mcpu=cortex-a76 -march=armv8.2-a
cflags: -DFORCE_AARCH64
- name: Graviton3
ec2_instance_type: c7g.medium
ec2_ami: ubuntu-latest (aarch64)
archflags: -march=armv8.4-a+sha3
cflags: -DFORCE_AARCH64
uses: ./.github/workflows/bench_ec2_reusable.yml
if: github.repository_owner == 'pq-code-package' && (github.event.label.name == 'benchmark' || github.ref == 'refs/heads/main')
with:
ec2_instance_type: ${{ matrix.target.ec2_instance_type }}
ec2_ami: ${{ matrix.target.ec2_ami }}
archflags: ${{ matrix.target.archflags }}
cflags: ${{ matrix.target.cflags }}
opt: ${{ matrix.opt.value }}
store_results: ${{ matrix.opt.value && github.repository_owner == 'pq-code-package' && github.ref == 'refs/heads/main' }} # Only store optimized results
name: ${{ matrix.target.name }}
secrets: inherit
46 changes: 0 additions & 46 deletions .github/workflows/bench_ec2_all.yml

This file was deleted.

0 comments on commit b546dea

Please sign in to comment.