Skip to content

Commit c60a3a2

Browse files
[CI][WIN] Don't skip E2E tests if only check-* steps in build fail (#10101)
Same as was done in #9912 for Linux.
1 parent cd079b6 commit c60a3a2

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

.github/workflows/sycl_windows_build_and_test.yml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,8 @@ jobs:
2929
name: Build + LIT
3030
runs-on: [Windows, build]
3131
# TODO use cached checkout
32+
outputs:
33+
build_conclusion: ${{ steps.build.conclusion }}
3234
steps:
3335
- uses: ilammy/msvc-dev-cmd@7315a94840631165970262a99c72cfb48a65d25d
3436
with:
@@ -92,6 +94,7 @@ jobs:
9294
run: |
9395
cmake --build build --target check-libdevice
9496
- name: Install
97+
if: ${{ always() && !cancelled() && steps.build.conclusion == 'success' }}
9598
shell: bash
9699
# TODO replace utility installation with a single CMake target
97100
run: |
@@ -106,18 +109,21 @@ jobs:
106109
cmake --build build --target install-llvm-profdata
107110
cmake --build build --target install-compiler-rt
108111
- name: Pack toolchain
112+
if: ${{ always() && !cancelled() && steps.build.conclusion == 'success' }}
109113
shell: bash
110114
run: |
111115
tar -czf llvm_sycl.tar.gz -C install .
112116
- name: Upload toolchain
117+
if: ${{ always() && !cancelled() && steps.build.conclusion == 'success' }}
113118
uses: actions/upload-artifact@v3
114119
with:
115120
name: sycl_windows_default
116121
path: llvm_sycl.tar.gz
117122

118123
e2e-tests:
119124
needs: build
120-
if: ${{ inputs.lts_matrix != '[]' }}
125+
# Continue if build was successful.
126+
if: ${{ always() && needs.build.outputs.build_conclusion == 'success' && inputs.lts_matrix != '[]' }}
121127
strategy:
122128
fail-fast: false
123129
matrix:

0 commit comments

Comments
 (0)