Skip to content

Commit cf36f16

Browse files
committed
do not build and run E2E test on benchmark only changes
1 parent febf259 commit cf36f16

File tree

2 files changed

+18
-3
lines changed

2 files changed

+18
-3
lines changed

.github/workflows/sycl-detect-changes.yml

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@ jobs:
1616
timeout-minutes: 3
1717
outputs:
1818
filters: ${{ steps.result.outputs.result }}
19+
benchmarks_only: ${{ steps.benchmarks_only.outputs.result }}
1920
steps:
2021
- name: Set changed_files
2122
id: changed_files
@@ -103,5 +104,13 @@ jobs:
103104
// Treat everything as changed for huge PRs.
104105
return ["llvm", "llvm_spirv", "clang", "sycl_jit", "xptifw", "libclc", "sycl", "ci", "esimd", "ur", "ur_cuda_adapter", "ur_offload_adapter"];
105106
106-
- run: echo '${{ steps.result.outputs.result }}'
107+
- name: Set benchmarks_only output
108+
id: benchmarks_only
109+
env:
110+
CHANGES: ${{ steps.changes.outputs.changes }}
111+
run: |
112+
has_benchmarks_only=$(echo "$CHANGES" | jq '[.[] | select(. != "ci" and . != "benchmarks")] | length == 0')
113+
echo "has_benchmarks_only=$has_benchmarks_only" >> $GITHUB_OUTPUT
114+
115+
- run: echo '${{ steps.result.outputs.result }} ${{ steps.benchmark_only.outputs.result }}'
107116

.github/workflows/sycl-linux-precommit.yml

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,10 @@ jobs:
6565
# Build and run native cpu e2e tests separately as cannot currently
6666
# build all the e2e tests
6767
build_run_native_cpu_e2e_tests:
68-
if: ${{ always() && !cancelled() && needs.build.outputs.build_conclusion == 'success' }}
68+
if: |
69+
always() && !cancelled()
70+
&& needs.build.outputs.build_conclusion == 'success'
71+
&& needs.detect_changes.outputs.benchmarks_only == 'false'
6972
runs-on: [Linux, build]
7073
needs: [build]
7174
container:
@@ -138,7 +141,10 @@ jobs:
138141
139142
E2E:
140143
needs: [build, detect_changes, compat_read_exclude]
141-
if: ${{ always() && !cancelled() && needs.build.outputs.build_conclusion == 'success' }}
144+
if: |
145+
always() && !cancelled()
146+
&& needs.build.outputs.build_conclusion == 'success'
147+
&& needs.detect_changes.outputs.benchmarks_only == 'false'
142148
strategy:
143149
fail-fast: false
144150
matrix:

0 commit comments

Comments
 (0)