Skip to content

Commit 066e266

Browse files
[CI][Bench] Add names for steps in composite action (#20498)
When action is used as composite, its steps don't disaply the actual 'name' defined for each step. Instead a full command to execute is displayed. To work around it we can put name as a comment in the first line of the command.
1 parent 6b4e685 commit 066e266

File tree

2 files changed

+11
-3
lines changed

2 files changed

+11
-3
lines changed

.github/workflows/sycl-linux-run-tests.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -330,7 +330,8 @@ jobs:
330330
- run: which clang++ sycl-ls
331331
- run: sycl-ls --verbose
332332
- run: SYCL_UR_TRACE=1 sycl-ls
333-
- run: |
333+
- name: Print IGC version
334+
run: |
334335
if [ -f /usr/local/lib/igc/IGCTAG.txt ]; then
335336
cat /usr/local/lib/igc/IGCTAG.txt
336337
fi

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

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@ inputs:
3838
required: False
3939

4040
runs:
41+
# composite actions don't make use of 'name', so copy-paste names as a comment in the first line of each step
4142
using: "composite"
4243
steps:
4344
- name: Check specified runner type / target backend
@@ -46,6 +47,7 @@ runs:
4647
TARGET_DEVICE: ${{ inputs.target_devices }}
4748
PRESET: ${{ inputs.preset }}
4849
run: |
50+
# Check specified runner type / target backend
4951
case "$RUNNER_TAG" in
5052
'["PVC_PERF"]' ) ;;
5153
'["BMG_PERF"]' ) ;;
@@ -114,6 +116,7 @@ runs:
114116
- name: Build Unified Runtime
115117
shell: bash
116118
run: |
119+
# Build Unified Runtime
117120
# Sparse-checkout UR at build ref:
118121
git clone --depth 1 --no-checkout https://github.com/intel/llvm ur
119122
cd ur
@@ -140,14 +143,16 @@ runs:
140143
- name: Checkout results repo
141144
shell: bash
142145
run: |
146+
# Checkout results repo
143147
git clone -b unify-ci https://github.com/intel/llvm-ci-perf-results
144-
- name: Run compute-benchmarks
148+
- name: Build and run benchmarks
145149
env:
146150
# Need to append "_<device>_<backend>" to save name in order to follow
147151
# conventions:
148152
SAVE_PREFIX: ${{ inputs.save_name }}
149153
shell: bash
150154
run: |
155+
# Build and run benchmarks
151156
# TODO generate summary + display helpful message here
152157
export CMPLR_ROOT=./toolchain
153158
echo "-----"
@@ -231,7 +236,8 @@ runs:
231236
- name: Cache changes and upload github summary
232237
if: always()
233238
shell: bash
234-
run: |
239+
run: |
240+
# Cache changes and upload github summary
235241
[ -f "github_summary.md" ] && cat github_summary.md >> $GITHUB_STEP_SUMMARY
236242
237243
cd "./llvm-ci-perf-results"
@@ -244,6 +250,7 @@ runs:
244250
if: inputs.upload_results == 'true' && always()
245251
shell: bash
246252
run: |
253+
# Push benchmarks results
247254
cd "./llvm-ci-perf-results"
248255
git config user.name "SYCL Benchmarking Bot"
249256
git config user.email "sys_sycl_benchmarks@intel.com"

0 commit comments

Comments
 (0)