From 6d60be993c4418e886d81eeb738e584a778500a5 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C5=81ukasz=20=C5=9Alusarczyk?= Date: Tue, 21 Oct 2025 12:34:39 +0200 Subject: [PATCH 1/2] [CI] do not build and run E2E test on benchmark only changes --- .github/workflows/sycl-detect-changes.yml | 7 ++++--- .github/workflows/sycl-linux-precommit.yml | 14 ++++++++++---- .github/workflows/sycl-windows-precommit.yml | 1 + 3 files changed, 15 insertions(+), 7 deletions(-) diff --git a/.github/workflows/sycl-detect-changes.yml b/.github/workflows/sycl-detect-changes.yml index 281e60db073b7..baeeb8e366903 100644 --- a/.github/workflows/sycl-detect-changes.yml +++ b/.github/workflows/sycl-detect-changes.yml @@ -46,7 +46,7 @@ jobs: - *llvm_spirv - *clang - 'libclc/**' - sycl: &sycl + sycl: - *clang - *sycl-jit - *llvm_spirv @@ -67,6 +67,8 @@ jobs: - devops/dependencies-igc-dev.json benchmarks: - 'devops/scripts/benchmarks/**' + non_benchmarks: + - '!devops/scripts/benchmarks/**' perf-tests: - sycl/test-e2e/PerformanceTests/** esimd: @@ -101,7 +103,6 @@ jobs: return '${{ steps.changes.outputs.changes }}'; } // Treat everything as changed for huge PRs. - return ["llvm", "llvm_spirv", "clang", "sycl_jit", "xptifw", "libclc", "sycl", "ci", "esimd", "ur", "ur_cuda_adapter", "ur_offload_adapter"]; + 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"]; - run: echo '${{ steps.result.outputs.result }}' - diff --git a/.github/workflows/sycl-linux-precommit.yml b/.github/workflows/sycl-linux-precommit.yml index 12986e98617fc..e253885f67aec 100644 --- a/.github/workflows/sycl-linux-precommit.yml +++ b/.github/workflows/sycl-linux-precommit.yml @@ -59,13 +59,16 @@ jobs: changes: ${{ needs.detect_changes.outputs.filters }} toolchain_artifact: sycl_linux_default - e2e_binaries_artifact: e2e_bin - e2e_binaries_preview_artifact: e2e_bin_preview + e2e_binaries_artifact: ${{ contains(needs.detect_changes.outputs.filters, 'non_benchmarks') && 'e2e_bin' || '' }} + e2e_binaries_preview_artifact: ${{ contains(needs.detect_changes.outputs.filters, 'non_benchmarks') && 'e2e_bin_preview' || '' }} # Build and run native cpu e2e tests separately as cannot currently # build all the e2e tests build_run_native_cpu_e2e_tests: - if: ${{ always() && !cancelled() && needs.build.outputs.build_conclusion == 'success' }} + if: | + always() && !cancelled() + && needs.build.outputs.build_conclusion == 'success' + && contains(needs.detect_changes.outputs.filters, 'non_benchmarks') runs-on: [Linux, build] needs: [build] container: @@ -138,7 +141,10 @@ jobs: E2E: needs: [build, detect_changes, compat_read_exclude] - if: ${{ always() && !cancelled() && needs.build.outputs.build_conclusion == 'success' }} + if: | + always() && !cancelled() + && needs.build.outputs.build_conclusion == 'success' + && contains(needs.detect_changes.outputs.filters, 'non_benchmarks') strategy: fail-fast: false matrix: diff --git a/.github/workflows/sycl-windows-precommit.yml b/.github/workflows/sycl-windows-precommit.yml index b8f8c675151d2..063e21337ea52 100644 --- a/.github/workflows/sycl-windows-precommit.yml +++ b/.github/workflows/sycl-windows-precommit.yml @@ -26,6 +26,7 @@ on: - 'devops/containers/**' - 'devops/actions/build_container/**' - 'devops/compat_ci_exclude.sycl-rel-6_2' + - 'devops/scripts/benchmarks/**' - 'unified-runtime/examples/**' - 'unified-runtime/scripts/**' - 'unified-runtime/test/**' From ff3fb0e52ae9dc670850381bc86296c2a351d130 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C5=81ukasz=20=C5=9Alusarczyk?= Date: Fri, 24 Oct 2025 09:23:02 +0200 Subject: [PATCH 2/2] renamed non_benchmarks to nonbench --- .github/workflows/sycl-detect-changes.yml | 4 ++-- .github/workflows/sycl-linux-precommit.yml | 8 ++++---- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/.github/workflows/sycl-detect-changes.yml b/.github/workflows/sycl-detect-changes.yml index baeeb8e366903..3ef13fd4e7840 100644 --- a/.github/workflows/sycl-detect-changes.yml +++ b/.github/workflows/sycl-detect-changes.yml @@ -67,7 +67,7 @@ jobs: - devops/dependencies-igc-dev.json benchmarks: - 'devops/scripts/benchmarks/**' - non_benchmarks: + nonbench: - '!devops/scripts/benchmarks/**' perf-tests: - sycl/test-e2e/PerformanceTests/** @@ -103,6 +103,6 @@ jobs: return '${{ steps.changes.outputs.changes }}'; } // Treat everything as changed for huge PRs. - 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"]; + return ["llvm", "llvm_spirv", "clang", "sycl_jit", "xptifw", "libclc", "sycl", "ci", "drivers", "devigccfg", "benchmarks", "nonbench", "perf-tests", "esimd", "ur", "ur_cuda_adapter", "ur_offload_adapter"]; - run: echo '${{ steps.result.outputs.result }}' diff --git a/.github/workflows/sycl-linux-precommit.yml b/.github/workflows/sycl-linux-precommit.yml index e253885f67aec..8e99a8c51c2a1 100644 --- a/.github/workflows/sycl-linux-precommit.yml +++ b/.github/workflows/sycl-linux-precommit.yml @@ -59,8 +59,8 @@ jobs: changes: ${{ needs.detect_changes.outputs.filters }} toolchain_artifact: sycl_linux_default - e2e_binaries_artifact: ${{ contains(needs.detect_changes.outputs.filters, 'non_benchmarks') && 'e2e_bin' || '' }} - e2e_binaries_preview_artifact: ${{ contains(needs.detect_changes.outputs.filters, 'non_benchmarks') && 'e2e_bin_preview' || '' }} + e2e_binaries_artifact: ${{ contains(needs.detect_changes.outputs.filters, 'nonbench') && 'e2e_bin' || '' }} + e2e_binaries_preview_artifact: ${{ contains(needs.detect_changes.outputs.filters, 'nonbench') && 'e2e_bin_preview' || '' }} # Build and run native cpu e2e tests separately as cannot currently # build all the e2e tests @@ -68,7 +68,7 @@ jobs: if: | always() && !cancelled() && needs.build.outputs.build_conclusion == 'success' - && contains(needs.detect_changes.outputs.filters, 'non_benchmarks') + && contains(needs.detect_changes.outputs.filters, 'nonbench') runs-on: [Linux, build] needs: [build] container: @@ -144,7 +144,7 @@ jobs: if: | always() && !cancelled() && needs.build.outputs.build_conclusion == 'success' - && contains(needs.detect_changes.outputs.filters, 'non_benchmarks') + && contains(needs.detect_changes.outputs.filters, 'nonbench') strategy: fail-fast: false matrix: