Skip to content

Commit c922ac7

Browse files
committed
[TEST] Test new runner
1 parent 41a107a commit c922ac7

File tree

3 files changed

+21
-11
lines changed

3 files changed

+21
-11
lines changed

.github/workflows/sycl-ur-perf-benchmarking.yml

Lines changed: 19 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,10 @@
11
name: Run Benchmarks
22

33
on:
4+
pull_request:
5+
types: [opened, synchronize, reopened, ready_for_review]
6+
branches:
7+
- sycl
48
workflow_call:
59
inputs:
610
preset:
@@ -35,18 +39,21 @@ on:
3539
results, this will be the name used to refer results from the current
3640
run.
3741
required: false
38-
default: ''
42+
default: 'test_runner_46'
3943
upload_results:
4044
type: string # true/false: workflow_dispatch does not support booleans
4145
description: |
4246
Upload results to https://intel.github.io/llvm/benchmarks/.
43-
required: true
47+
required: false
48+
default: 'true'
4449
runner:
4550
type: string
46-
required: true
51+
required: false
52+
default: '["test-runner"]'
4753
backend:
4854
type: string
49-
required: true
55+
required: false
56+
default: 'level_zero_v2:gpu'
5057

5158
workflow_dispatch:
5259
inputs:
@@ -93,14 +100,17 @@ on:
93100
runner:
94101
type: choice
95102
options:
103+
- '["test-runner"]'
96104
- '["PVC_PERF"]'
97105
- '["BMG_PERF"]'
106+
default: '["test-runner"]'
98107
backend:
99108
description: Backend to use
100109
type: choice
101110
options:
102-
- 'level_zero:gpu'
103111
- 'level_zero_v2:gpu'
112+
- 'level_zero:gpu'
113+
default: 'level_zero_v2:gpu'
104114
# As of #17407, sycl-linux-build now builds v2 by default
105115

106116
permissions: read-all
@@ -110,7 +120,7 @@ jobs:
110120
name: Sanitize inputs
111121
runs-on: ubuntu-latest
112122
env:
113-
COMMIT_HASH: ${{ inputs.commit_hash }}
123+
COMMIT_HASH: ${{ inputs.commit_hash || }}
114124
PR_NO: ${{ inputs.pr_no }}
115125
SAVE_NAME: ${{ inputs.save_name }}
116126
outputs:
@@ -178,12 +188,12 @@ jobs:
178188
matrix:
179189
include:
180190
# Set default values if not specified:
181-
- runner: ${{ inputs.runner || '["PVC_PERF"]' }}
182-
backend: ${{ inputs.backend || 'level_zero:gpu' }}
191+
- runner: ${{ inputs.runner || '["test-runner"]' }}
192+
backend: ${{ inputs.backend || 'level_zero_v2:gpu' }}
183193
uses: ./.github/workflows/sycl-linux-run-tests.yml
184194
secrets: inherit
185195
with:
186-
name: Run compute-benchmarks (${{ matrix.save_name }}, ${{ matrix.runner }}, ${{ matrix.backend }})
196+
name: Run compute-benchmarks (${{ needs.sanitize_inputs.outputs.benchmark_save_name }}, ${{ matrix.runner }}, ${{ matrix.backend }})
187197
runner: ${{ matrix.runner }}
188198
image: ghcr.io/intel/llvm/sycl_ubuntu2404_nightly:latest
189199
image_options: -u 1001 --device=/dev/dri -v /dev/dri/by-path:/dev/dri/by-path --privileged --cap-add SYS_ADMIN

devops/actions/run-tests/benchmark/action.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -172,7 +172,7 @@ runs:
172172
esac
173173
case "$RUNNER_TAG" in
174174
'["PVC_PERF"]') MACHINE_TYPE="PVC" ;;
175-
'["BMG_PERF"]') MACHINE_TYPE="BMG" ;;
175+
'["test-runner2"]') MACHINE_TYPE="BMG" ;;
176176
# Best effort at matching
177177
*)
178178
MACHINE_TYPE="${RUNNER_TAG#[\"}"

devops/scripts/benchmarks/utils/validate.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ def runner_name(runner_name: str, throw: Exception = None):
2727
"""
2828
Returns True if runner_name is clean (no illegal characters).
2929
"""
30-
return Validate.on_re(runner_name, r"^[a-zA-Z0-9_]+$", throw=throw)
30+
return Validate.on_re(runner_name, r"^[a-zA-Z0-9_-]+$", throw=throw)
3131

3232
@staticmethod
3333
def timestamp(t: str, throw: Exception = None):

0 commit comments

Comments
 (0)