Skip to content

Commit 6d60be9

Browse files
committed
[CI] do not build and run E2E test on benchmark only changes
1 parent e9248dd commit 6d60be9

File tree

3 files changed

+15
-7
lines changed

3 files changed

+15
-7
lines changed

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

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ jobs:
4646
- *llvm_spirv
4747
- *clang
4848
- 'libclc/**'
49-
sycl: &sycl
49+
sycl:
5050
- *clang
5151
- *sycl-jit
5252
- *llvm_spirv
@@ -67,6 +67,8 @@ jobs:
6767
- devops/dependencies-igc-dev.json
6868
benchmarks:
6969
- 'devops/scripts/benchmarks/**'
70+
non_benchmarks:
71+
- '!devops/scripts/benchmarks/**'
7072
perf-tests:
7173
- sycl/test-e2e/PerformanceTests/**
7274
esimd:
@@ -101,7 +103,6 @@ jobs:
101103
return '${{ steps.changes.outputs.changes }}';
102104
}
103105
// Treat everything as changed for huge PRs.
104-
return ["llvm", "llvm_spirv", "clang", "sycl_jit", "xptifw", "libclc", "sycl", "ci", "esimd", "ur", "ur_cuda_adapter", "ur_offload_adapter"];
106+
return ["llvm", "llvm_spirv", "clang", "sycl_jit", "xptifw", "libclc", "sycl", "ci", "drivers", "devigccfg", "benchmarks", "non_benchmarks", "perf-tests", "esimd", "ur", "ur_cuda_adapter", "ur_offload_adapter"];
105107
106108
- run: echo '${{ steps.result.outputs.result }}'
107-

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

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -59,13 +59,16 @@ jobs:
5959
changes: ${{ needs.detect_changes.outputs.filters }}
6060

6161
toolchain_artifact: sycl_linux_default
62-
e2e_binaries_artifact: e2e_bin
63-
e2e_binaries_preview_artifact: e2e_bin_preview
62+
e2e_binaries_artifact: ${{ contains(needs.detect_changes.outputs.filters, 'non_benchmarks') && 'e2e_bin' || '' }}
63+
e2e_binaries_preview_artifact: ${{ contains(needs.detect_changes.outputs.filters, 'non_benchmarks') && 'e2e_bin_preview' || '' }}
6464

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+
&& contains(needs.detect_changes.outputs.filters, 'non_benchmarks')
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+
&& contains(needs.detect_changes.outputs.filters, 'non_benchmarks')
142148
strategy:
143149
fail-fast: false
144150
matrix:

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

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@ on:
2626
- 'devops/containers/**'
2727
- 'devops/actions/build_container/**'
2828
- 'devops/compat_ci_exclude.sycl-rel-6_2'
29+
- 'devops/scripts/benchmarks/**'
2930
- 'unified-runtime/examples/**'
3031
- 'unified-runtime/scripts/**'
3132
- 'unified-runtime/test/**'

0 commit comments

Comments
 (0)