From 59f480bdaecd28bdc65c5e7e5451b30f3ed28d80 Mon Sep 17 00:00:00 2001 From: "Sarnie, Nick" Date: Mon, 17 Mar 2025 09:17:46 -0700 Subject: [PATCH 1/2] [CI] Correctly set install_igc_driver and related opts in postcommit Signed-off-by: Sarnie, Nick --- .github/workflows/sycl-post-commit.yml | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/.github/workflows/sycl-post-commit.yml b/.github/workflows/sycl-post-commit.yml index 9234910898bc1..ac763d5372ee3 100644 --- a/.github/workflows/sycl-post-commit.yml +++ b/.github/workflows/sycl-post-commit.yml @@ -29,6 +29,9 @@ concurrency: permissions: read-all jobs: + detect_changes: + uses: ./.github/workflows/sycl-detect-changes.yml + build-lin: name: Linux (Self build + no-assertions) if: github.repository == 'intel/llvm' @@ -40,7 +43,7 @@ jobs: build_configure_extra_args: --no-assertions --hip --cuda --native_cpu -DSYCL_ENABLE_STACK_PRINTING=ON -DSYCL_LIB_WITH_DEBUG_SYMBOL=ON e2e-lin: - needs: [build-lin] + needs: [detect_changes, build-lin] if: ${{ always() && !cancelled() && needs.build-lin.outputs.build_conclusion == 'success' }} strategy: fail-fast: false @@ -94,6 +97,12 @@ jobs: sycl_toolchain_archive: ${{ needs.build-lin.outputs.artifact_archive_name }} sycl_toolchain_decompress_command: ${{ needs.build-lin.outputs.artifact_decompress_command }} + # Do not install drivers on AMD and CUDA runners. + install_igc_driver: >- + ${{ !contains(matrix.target_devices, 'cuda') && + !contains(matrix.target_devices, 'hip') && + contains(needs.detect_changes.outputs.filters, 'drivers') }} + build-win: if: | always() From 5b3d349e30b526514d629d1fee1fdc0f1dbb4e57 Mon Sep 17 00:00:00 2001 From: "Sarnie, Nick" Date: Mon, 17 Mar 2025 14:47:51 -0700 Subject: [PATCH 2/2] fix maybe Signed-off-by: Sarnie, Nick --- .github/workflows/sycl-post-commit.yml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.github/workflows/sycl-post-commit.yml b/.github/workflows/sycl-post-commit.yml index ac763d5372ee3..6c97dedf21fc2 100644 --- a/.github/workflows/sycl-post-commit.yml +++ b/.github/workflows/sycl-post-commit.yml @@ -30,6 +30,7 @@ permissions: read-all jobs: detect_changes: + if: ${{ github.event_name == 'pull_request' }} uses: ./.github/workflows/sycl-detect-changes.yml build-lin: @@ -99,6 +100,7 @@ jobs: # Do not install drivers on AMD and CUDA runners. install_igc_driver: >- + ${{ github.event_name == 'pull_request' }} && ${{ !contains(matrix.target_devices, 'cuda') && !contains(matrix.target_devices, 'hip') && contains(needs.detect_changes.outputs.filters, 'drivers') }}