Skip to content

Commit

Permalink
Use new running scripts in ci-perf-kit (#1005)
Browse files Browse the repository at this point in the history
This PR updates to ci-perf-kit 0.7.0. It fixes a few issues:
* Use new running scripts: #980
* Run all the benchmarks for all the plans:
#981
* Perf comparison for PR works again:
#906
  • Loading branch information
qinsoon authored Nov 3, 2023
1 parent 39d2371 commit dad49c2
Show file tree
Hide file tree
Showing 4 changed files with 14 additions and 9 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/micro-bm.yml
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ jobs:
with:
repository: mmtk/ci-perf-kit
token: ${{ secrets.GITHUB_TOKEN }}
ref: "0.6.8"
ref: "0.7.0"
path: ci-perf-kit
submodules: true
# Use rust-toolchain in the trunk (it doesnt matter much - if the toolchains defined in the trunk and the branch are different, we cant run anyway)
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/perf-baseline.yml
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ jobs:
with:
token: ${{ secrets.CI_ACCESS_TOKEN }}
repository: mmtk/ci-perf-kit
ref: "0.6.10"
ref: "0.7.0"
path: ci-perf-kit
submodules: true
# setup
Expand Down Expand Up @@ -79,7 +79,7 @@ jobs:
with:
token: ${{ secrets.CI_ACCESS_TOKEN }}
repository: mmtk/ci-perf-kit
ref: "0.6.10"
ref: "0.7.0"
path: ci-perf-kit
submodules: true
# setup
Expand Down
8 changes: 5 additions & 3 deletions .github/workflows/perf-compare-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,7 @@ jobs:
jikesrvm-perf-compare:
runs-on: [self-hosted, Linux, freq-scaling-off]
needs: [binding-refs, mmtk-refs]
timeout-minutes: 1440
steps:
# Trunk - we always use master from the mmtk org
# - binding
Expand Down Expand Up @@ -106,7 +107,7 @@ jobs:
with:
repository: mmtk/ci-perf-kit
token: ${{ secrets.CI_ACCESS_TOKEN }}
ref: "0.6.8"
ref: "0.7.0"
path: ci-perf-kit
submodules: true
# setup
Expand All @@ -129,7 +130,7 @@ jobs:
- name: Compare Performance
id: run
run: |
JAVA_HOME=/usr/lib/jvm/java-1.8.0-openjdk-amd64 ./ci-perf-kit/scripts/jikesrvm-compare.sh mmtk-jikesrvm-trunk/ mmtk-core-trunk/ mmtk-jikesrvm-branch/ mmtk-core-branch/ jikesrvm-compare-report.md
JAVA_HOME=/opt/jdk1.6.0_45/ PATH=/opt/apache-ant-1.9.16/bin/:/opt/jdk1.6.0_45/bin/:$PATH ./ci-perf-kit/scripts/jikesrvm-compare.sh mmtk-jikesrvm-trunk/ mmtk-core-trunk/ mmtk-jikesrvm-branch/ mmtk-core-branch/ jikesrvm-compare-report.md
# set report.md to output
- uses: pCYSl5EDgo/cat@master
id: cat
Expand Down Expand Up @@ -165,6 +166,7 @@ jobs:
openjdk-perf-compare:
runs-on: [self-hosted, Linux, freq-scaling-off]
needs: [binding-refs, mmtk-refs]
timeout-minutes: 1440
steps:
# Trunk - we always use master from the mmtk org
# - binding
Expand Down Expand Up @@ -208,7 +210,7 @@ jobs:
with:
repository: mmtk/ci-perf-kit
token: ${{ secrets.CI_ACCESS_TOKEN }}
ref: "0.6.8"
ref: "0.7.0"
path: ci-perf-kit
submodules: true
# setup
Expand Down
9 changes: 6 additions & 3 deletions .github/workflows/perf-regression-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@ jobs:
# JikesRVM
jikesrvm-perf-regression:
runs-on: [self-hosted, Linux, freq-scaling-off]
timeout-minutes: 1440
steps:
- name: Checkout MMTk Core
uses: actions/checkout@v2
Expand All @@ -41,7 +42,7 @@ jobs:
uses: actions/checkout@v2
with:
repository: mmtk/ci-perf-kit
ref: "0.6.10"
ref: "0.7.0"
path: ci-perf-kit
token: ${{ secrets.CI_ACCESS_TOKEN }}
submodules: true
Expand Down Expand Up @@ -82,6 +83,7 @@ jobs:
# OpenJDK
openjdk-perf-regression:
runs-on: [self-hosted, Linux, freq-scaling-off]
timeout-minutes: 1440
steps:
- name: Checkout MMTk Core
uses: actions/checkout@v2
Expand All @@ -101,7 +103,7 @@ jobs:
uses: actions/checkout@v2
with:
repository: mmtk/ci-perf-kit
ref: "0.6.10"
ref: "0.7.0"
path: ci-perf-kit
token: ${{ secrets.CI_ACCESS_TOKEN }}
submodules: true
Expand Down Expand Up @@ -146,6 +148,7 @@ jobs:

openjdk-mutator-perf:
runs-on: [self-hosted, Linux, freq-scaling-off]
timeout-minutes: 1440
steps:
- name: Checkout MMTk Core
uses: actions/checkout@v2
Expand All @@ -165,7 +168,7 @@ jobs:
uses: actions/checkout@v2
with:
repository: mmtk/ci-perf-kit
ref: "0.6.10"
ref: "0.7.0"
path: ci-perf-kit
token: ${{ secrets.CI_ACCESS_TOKEN }}
submodules: true
Expand Down

0 comments on commit dad49c2

Please sign in to comment.