Skip to content

[CI] Refactor cleanup of built E2E tests in precommit. #20790

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 7 commits into from
Dec 4, 2025
+9 −8
Merged

[CI] Refactor cleanup of built E2E tests in precommit. #20790

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
20d47f0
[CI] Refactor cleanup of built E2E tests in precommit.
maksimsab Dec 1, 2025
cdb2b1f
fix type and add matrix
maksimsab Dec 1, 2025
cb6da22
fix typo
maksimsab Dec 1, 2025
0ea1b34
fix shell and binaries_artifact path
maksimsab Dec 1, 2025
22eed78
revert matrix approach for building e2e tests
maksimsab Dec 2, 2025
5b6e2f3
fix shell in action.yml
maksimsab Dec 2, 2025
d461048
Merge branch 'sycl' into refactor_ci
maksimsab Dec 3, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 0 additions & 8 deletions .github/workflows/sycl-linux-build.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -339,10 +339,6 @@ jobs:
sycl_compiler: $GITHUB_WORKSPACE/toolchain/bin/clang++
extra_lit_opts: --param sycl_build_targets="spir;nvidia;amd"

- name: Remove E2E tests before spirv-backend run
if: ${{ inputs.e2e_binaries_spirv_backend_artifact && !cancelled() && steps.build.conclusion == 'success' }}
run: rm -rf build-e2e

- name: Build E2E tests with SPIR-V Backend
if: ${{ inputs.e2e_binaries_spirv_backend_artifact && !cancelled() && steps.build.conclusion == 'success' }}
uses: ./devops/actions/run-tests/e2e
Expand All @@ -354,10 +350,6 @@ jobs:
sycl_compiler: $GITHUB_WORKSPACE/toolchain/bin/clang++
extra_lit_opts: --param spirv-backend=True

- name: Remove E2E tests before preview-mode run
if: ${{ inputs.e2e_binaries_preview_artifact && !cancelled() && steps.build.conclusion == 'success' }}
run: rm -rf build-e2e

- name: Build E2E tests in Preview Mode
if: ${{ inputs.e2e_binaries_preview_artifact && !cancelled() && steps.build.conclusion == 'success' }}
uses: ./devops/actions/run-tests/e2e
Expand Down
5 changes: 5 additions & 0 deletions devops/actions/run-tests/e2e/action.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -106,3 +106,8 @@ runs:
name: ${{ inputs.binaries_artifact }}
path: e2e_binaries.tar.zst
retention-days: ${{ inputs.retention-days }}
- name: Cleanup E2E tests
if: ${{ !cancelled() }}
shell: bash
run: |
rm -rf build-e2e
4 changes: 4 additions & 0 deletions devops/actions/run-tests/windows/e2e/action.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -121,3 +121,7 @@ runs:
name: ${{ inputs.binaries_artifact }}
path: e2e_bin.tar.gz
retention-days: ${{ inputs.retention-days }}
- name: Cleanup E2E tests
if: ${{ !cancelled() }}
shell: bash
run: rm -rf build-e2e
Loading