Skip to content

Commit f4fca6e

Browse files
authored
[CI] Remove workarounds (#19017)
devops_ref and cts_exclude_ref were used as workaronds to launch sycl-rel-nightly with the sycl-rel branch using the regular sycl branch. Now we run sycl-rel-nightly using the sycl-rel branch, so there is no need to keep these workarounds.
1 parent 1cba3bb commit f4fca6e

File tree

6 files changed

+7
-278
lines changed

6 files changed

+7
-278
lines changed

.github/workflows/sycl-linux-run-tests.yml

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -40,11 +40,6 @@ on:
4040
required: True
4141
description: |
4242
Commit SHA or branch to checkout the intel/llvm repo.
43-
devops_ref:
44-
type: string
45-
required: False
46-
description: |
47-
Commit SHA or branch to checkout the devops directory.
4843
tests_ref:
4944
type: string
5045
required: False
@@ -217,9 +212,9 @@ jobs:
217212
steps:
218213
- uses: actions/checkout@v4
219214
with:
220-
ref: ${{ inputs.devops_ref || inputs.repo_ref }}
221215
sparse-checkout: |
222216
devops
217+
sycl/cts_exclude_filter
223218
- name: Register cleanup after job is finished
224219
uses: ./devops/actions/cleanup
225220
- name: Reset Intel GPU
@@ -326,7 +321,6 @@ jobs:
326321
timeout-minutes: 35
327322
with:
328323
ref: ${{ inputs.tests_ref || 'main' }}
329-
cts_exclude_ref: ${{ inputs.repo_ref }}
330324
extra_cmake_args: ${{ inputs.extra_cmake_args }}
331325
cts_testing_mode: ${{ inputs.cts_testing_mode }}
332326
sycl_cts_artifact: ${{ inputs.sycl_cts_artifact }}
Lines changed: 5 additions & 233 deletions
Original file line numberDiff line numberDiff line change
@@ -1,241 +1,13 @@
1+
# The actual version of this workflow can be found on the latest sycl-rel-*
2+
# branch. This file is still needed to launch the workflow on another branches.
3+
14
name: SYCL Release Branch Nightly
25

36
on:
47
workflow_dispatch:
5-
inputs:
6-
testing_branch:
7-
description: |
8-
Branch to test, e.g. sycl-rel-6_0_0.
9-
If nothing is specified, the last release branch is used.
10-
11-
schedule:
12-
- cron: '0 3 * * *'
13-
14-
permissions: read-all
158

169
jobs:
17-
# To avoid unnecessary scheduled runs this job checks if there are new commits
18-
# since the last run. More precisely, it checks if the last commit is older
19-
# than 24h. That means the previous Nightly already tested this commit.
20-
check_for_new_commits:
21-
if: github.repository == 'intel/llvm'
10+
dummy:
2211
runs-on: ubuntu-latest
23-
name: Check for new commits
24-
outputs:
25-
is_new_commit: ${{ steps.is_new_commit.outputs.is_new_commit }}
2612
steps:
27-
- uses: actions/checkout@v4
28-
with:
29-
ref: ${{ inputs.testing_branch || 'sycl-rel-6_2' }}
30-
- run: git show --quiet | tee -a $GITHUB_STEP_SUMMARY
31-
32-
- id: is_new_commit
33-
if: ${{ github.event_name == 'schedule' }}
34-
run: |
35-
if [ -z "$(git rev-list --after="24 hours" HEAD)" ]; then
36-
echo "is_new_commit=false" >> $GITHUB_OUTPUT
37-
fi
38-
39-
ubuntu2204_build:
40-
needs: [check_for_new_commits]
41-
if: ${{ github.repository == 'intel/llvm' && needs.check_for_new_commits.outputs.is_new_commit != 'false' }}
42-
uses: ./.github/workflows/sycl-linux-build.yml
43-
secrets: inherit
44-
with:
45-
build_cache_root: "/__w/"
46-
build_artifact_suffix: default
47-
build_configure_extra_args: '--disable-jit --no-assertions --add_security_flags=sanitize --hip --cuda'
48-
build_image: ghcr.io/intel/llvm/ubuntu2204_build:latest
49-
build_ref: ${{ inputs.testing_branch || 'sycl-rel-6_2' }}
50-
pack_release: 'true'
51-
52-
# We upload the build for people to download/use, override its name and
53-
# prefer widespread gzip compression.
54-
artifact_archive_name: sycl_linux.tar.gz
55-
56-
ubuntu2204_test:
57-
needs: [ubuntu2204_build]
58-
if: ${{ always() && !cancelled() && needs.ubuntu2204_build.outputs.build_conclusion == 'success' }}
59-
strategy:
60-
fail-fast: false
61-
matrix:
62-
include:
63-
- name: AMD/HIP
64-
runner: '["Linux", "amdgpu"]'
65-
image_options: -u 1001 --device=/dev/dri --device=/dev/kfd
66-
target_devices: hip:gpu
67-
68-
- name: Intel L0 Gen12 GPU
69-
runner: '["Linux", "gen12"]'
70-
image_options: -u 1001 --device=/dev/dri -v /dev/dri/by-path:/dev/dri/by-path --privileged --cap-add SYS_ADMIN
71-
target_devices: level_zero:gpu
72-
extra_lit_opts: --param gpu-intel-gen12=True
73-
74-
- name: Intel L0 Battlemage GPU
75-
runner: '["Linux", "bmg"]'
76-
image_options: -u 1001 --device=/dev/dri -v /dev/dri/by-path:/dev/dri/by-path --privileged --cap-add SYS_ADMIN
77-
target_devices: level_zero:gpu
78-
79-
- name: Intel OCL Gen12 GPU
80-
runner: '["Linux", "gen12"]'
81-
image_options: -u 1001 --device=/dev/dri -v /dev/dri/by-path:/dev/dri/by-path --privileged --cap-add SYS_ADMIN
82-
target_devices: opencl:gpu
83-
extra_lit_opts: --param gpu-intel-gen12=True
84-
85-
- name: Intel OCL CPU
86-
runner: '["Linux", "gen12"]'
87-
image_options: -u 1001 --privileged --cap-add SYS_ADMIN
88-
target_devices: opencl:cpu
89-
uses: ./.github/workflows/sycl-linux-run-tests.yml
90-
with:
91-
name: ${{ matrix.name }}
92-
runner: ${{ matrix.runner }}
93-
image_options: ${{ matrix.image_options }}
94-
target_devices: ${{ matrix.target_devices }}
95-
tests_selector: e2e
96-
extra_lit_opts: ${{ matrix.extra_lit_opts }}
97-
repo_ref: ${{ inputs.testing_branch || 'sycl-rel-6_2' }}
98-
devops_ref: sycl
99-
sycl_toolchain_artifact: sycl_linux_default
100-
sycl_toolchain_archive: ${{ needs.ubuntu2204_build.outputs.artifact_archive_name }}
101-
sycl_toolchain_decompress_command: ${{ needs.ubuntu2204_build.outputs.artifact_decompress_command }}
102-
103-
build-win:
104-
needs: [check_for_new_commits]
105-
if: ${{ github.repository == 'intel/llvm' && needs.check_for_new_commits.outputs.is_new_commit != 'false' }}
106-
uses: ./.github/workflows/sycl-windows-build.yml
107-
with:
108-
ref: ${{ inputs.testing_branch || 'sycl-rel-6_2' }}
109-
build_configure_extra_args: '--disable-jit --no-assertions --add_security_flags=sanitize'
110-
pack_release: 'true'
111-
112-
# We upload both Linux/Windows build via Github's "Releases"
113-
# functionality, make sure Linux/Windows names follow the same pattern.
114-
artifact_archive_name: sycl_windows.tar.gz
115-
116-
e2e-win:
117-
needs: build-win
118-
# Continue if build was successful.
119-
if: |
120-
always()
121-
&& !cancelled()
122-
&& needs.build-win.outputs.build_conclusion == 'success'
123-
strategy:
124-
fail-fast: false
125-
matrix:
126-
include:
127-
- name: Intel GEN12 Graphics with Level Zero
128-
runner: '["Windows","gen12"]'
129-
# 6_1_0 doesn't support the auto-detection, keep it until 6_2_0.
130-
extra_lit_opts: --param gpu-intel-gen12=True
131-
- name: Intel Battlemage Graphics with Level Zero
132-
runner: '["Windows","bmg"]'
133-
uses: ./.github/workflows/sycl-windows-run-tests.yml
134-
with:
135-
name: ${{ matrix.name }}
136-
runner: ${{ matrix.runner }}
137-
target_devices: level_zero:gpu
138-
sycl_toolchain_archive: ${{ needs.build-win.outputs.artifact_archive_name }}
139-
extra_lit_opts: ${{ matrix.extra_lit_opts }}
140-
repo_ref: ${{ inputs.testing_branch || 'sycl-rel-6_2' }}
141-
devops_ref: sycl
142-
143-
cuda-aws-start:
144-
needs: [ubuntu2204_build]
145-
if: ${{ always() && !cancelled() && needs.ubuntu2204_build.outputs.build_conclusion == 'success' }}
146-
uses: ./.github/workflows/sycl-aws.yml
147-
secrets: inherit
148-
with:
149-
mode: start
150-
ref: ${{ inputs.testing_branch || 'sycl-rel-6_2' }}
151-
152-
cuda-run-tests:
153-
needs: [ubuntu2204_build, cuda-aws-start]
154-
if: ${{ always() && !cancelled() && needs.ubuntu2204_build.outputs.build_conclusion == 'success' }}
155-
uses: ./.github/workflows/sycl-linux-run-tests.yml
156-
with:
157-
name: CUDA E2E
158-
runner: '["aws_cuda-${{ github.run_id }}-${{ github.run_attempt }}"]'
159-
image: ghcr.io/intel/llvm/ubuntu2204_build:latest
160-
image_options: -u 1001 --gpus all --cap-add SYS_ADMIN --env NVIDIA_DISABLE_REQUIRE=1
161-
target_devices: cuda:gpu
162-
repo_ref: ${{ inputs.testing_branch || 'sycl-rel-6_2' }}
163-
devops_ref: sycl
164-
165-
sycl_toolchain_artifact: sycl_linux_default
166-
sycl_toolchain_archive: ${{ needs.ubuntu2204_build.outputs.artifact_archive_name }}
167-
sycl_toolchain_decompress_command: ${{ needs.ubuntu2204_build.outputs.artifact_decompress_command }}
168-
169-
cuda-aws-stop:
170-
needs: [cuda-aws-start, cuda-run-tests]
171-
if: always() && ${{ needs.cuda-aws-start.result != 'skipped' }}
172-
uses: ./.github/workflows/sycl-aws.yml
173-
secrets: inherit
174-
with:
175-
mode: stop
176-
ref: ${{ inputs.testing_branch || 'sycl-rel-6_2' }}
177-
178-
build-sycl-cts:
179-
needs: ubuntu2204_build
180-
if: ${{ always() && !cancelled() && needs.ubuntu2204_build.outputs.build_conclusion == 'success' }}
181-
uses: ./.github/workflows/sycl-linux-run-tests.yml
182-
with:
183-
name: Build SYCL-CTS
184-
runner: '["Linux", "build"]'
185-
cts_testing_mode: 'build-only'
186-
image_options: -u 1001 --device=/dev/dri --privileged --cap-add SYS_ADMIN
187-
tests_selector: cts
188-
repo_ref: ${{ inputs.testing_branch || 'sycl-rel-6_2' }}
189-
devops_ref: sycl
190-
tests_ref: 87f8677c4b67cfe56ad6f09246036f10c306c977
191-
sycl_toolchain_artifact: sycl_linux_default
192-
sycl_toolchain_archive: ${{ needs.ubuntu2204_build.outputs.artifact_archive_name }}
193-
sycl_toolchain_decompress_command: ${{ needs.ubuntu2204_build.outputs.artifact_decompress_command }}
194-
sycl_cts_artifact: sycl_cts_bin_linux
195-
196-
run-sycl-cts:
197-
needs: [ubuntu2204_build, build-sycl-cts]
198-
if: ${{ always() && !cancelled() && needs.ubuntu2204_build.outputs.build_conclusion == 'success' }}
199-
strategy:
200-
fail-fast: false
201-
matrix:
202-
include:
203-
- name: SYCL-CTS on OCL CPU
204-
runner: '["Linux", "gen12"]'
205-
image_options: -u 1001 --device=/dev/dri --privileged --cap-add SYS_ADMIN
206-
target_devices: opencl:cpu
207-
208-
- name: SYCL-CTS on L0 gen12
209-
runner: '["Linux", "gen12"]'
210-
image_options: -u 1001 --device=/dev/dri --privileged --cap-add SYS_ADMIN
211-
target_devices: level_zero:gpu
212-
uses: ./.github/workflows/sycl-linux-run-tests.yml
213-
with:
214-
name: ${{ matrix.name }}
215-
runner: ${{ matrix.runner }}
216-
cts_testing_mode: 'run-only'
217-
image_options: ${{ matrix.image_options }}
218-
target_devices: ${{ matrix.target_devices }}
219-
tests_selector: cts
220-
repo_ref: ${{ inputs.testing_branch || 'sycl-rel-6_2' }}
221-
devops_ref: sycl
222-
sycl_toolchain_artifact: sycl_linux_default
223-
sycl_toolchain_archive: ${{ needs.ubuntu2204_build.outputs.artifact_archive_name }}
224-
sycl_toolchain_decompress_command: ${{ needs.ubuntu2204_build.outputs.artifact_decompress_command }}
225-
sycl_cts_artifact: sycl_cts_bin_linux
226-
227-
hardening-check:
228-
needs: [ubuntu2204_build, build-win]
229-
if: |
230-
always()
231-
&& !cancelled()
232-
&& needs.ubuntu2204_build.outputs.build_conclusion == 'success'
233-
&& needs.build-win.outputs.build_conclusion == 'success'
234-
uses: ./.github/workflows/sycl-hardening-check.yml
235-
with:
236-
sycl_linux_artifact: sycl_linux_release
237-
sycl_linux_archive: ${{ needs.ubuntu2204_build.outputs.artifact_archive_name }}
238-
sycl_linux_decompress_command: ${{ needs.ubuntu2204_build.outputs.artifact_decompress_command }}
239-
240-
sycl_windows_artifact: sycl_windows_release
241-
sycl_windows_archive: ${{ needs.build-win.outputs.artifact_archive_name }}
13+
- run: echo "Doing nothing"

.github/workflows/sycl-ur-perf-benchmarking.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -133,7 +133,6 @@ jobs:
133133
benchmark_save_name: ${{ matrix.save_name }}
134134
benchmark_preset: ${{ inputs.preset }}
135135
repo_ref: ${{ matrix.ref }}
136-
devops_ref: ${{ github.ref }}
137136
sycl_toolchain_artifact: sycl_linux_prod_noassert
138137
sycl_toolchain_archive: ${{ needs.build_sycl.outputs.artifact_archive_name }}
139138
sycl_toolchain_decompress_command: ${{ needs.build_sycl.outputs.artifact_decompress_command }}

.github/workflows/sycl-windows-run-tests.yml

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -34,11 +34,6 @@ on:
3434
required: False
3535
description: |
3636
Commit SHA or branch to checkout the intel/llvm repo.
37-
devops_ref:
38-
type: string
39-
required: False
40-
description: |
41-
Commit SHA or branch to checkout the devops directory.
4237
tests_ref:
4338
type: string
4439
required: False
@@ -111,7 +106,7 @@ jobs:
111106
with:
112107
sparse-checkout: |
113108
devops/actions
114-
ref: ${{ inputs.devops_ref|| inputs.repo_ref || github.sha }}
109+
sycl/cts_exclude_filter
115110
- uses: ilammy/msvc-dev-cmd@0b201ec74fa43914dc39ae48a89fd1d8cb592756
116111
with:
117112
arch: amd64
@@ -171,7 +166,6 @@ jobs:
171166
uses: ./devops/actions/run-tests/windows/cts
172167
with:
173168
ref: ${{ inputs.tests_ref || 'main' }}
174-
cts_exclude_ref: ${{ inputs.repo_ref }}
175169
extra_cmake_args: ${{ inputs.extra_cmake_args }}
176170
cts_testing_mode: ${{ inputs.cts_testing_mode }}
177171
sycl_cts_artifact: ${{ inputs.sycl_cts_artifact }}

devops/actions/run-tests/cts/action.yml

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,6 @@ inputs:
44
ref:
55
description: "Commit SHA or branch to checkout tests"
66
required: true
7-
cts_exclude_ref:
8-
description: "Commit SHA or branch to checkout the cts_exclude_filter dir"
9-
required: true
107
extra_cmake_args:
118
required: false
129
cts_testing_mode:
@@ -21,18 +18,6 @@ inputs:
2118
runs:
2219
using: "composite"
2320
steps:
24-
- name: Checkout cts_exclude_filter folder
25-
uses: actions/checkout@v4
26-
with:
27-
ref: ${{ inputs.cts_exclude_ref }}
28-
path: cts_exclude
29-
sparse-checkout: |
30-
sycl/cts_exclude_filter
31-
- name: Move sycl to root
32-
shell: bash
33-
run: |
34-
mv cts_exclude/sycl .
35-
rm -rf cts_exclude
3621
- name: Checkout SYCL CTS tests
3722
if: inputs.cts_testing_mode != 'run-only'
3823
uses: ./devops/actions/cached_checkout

devops/actions/run-tests/windows/cts/action.yml

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,6 @@ inputs:
44
ref:
55
description: "Commit SHA or branch to checkout tests"
66
required: true
7-
cts_exclude_ref:
8-
description: "Commit SHA or branch to checkout the cts_exclude_filter dir"
9-
required: true
107
extra_cmake_args:
118
required: false
129
cts_testing_mode:
@@ -21,18 +18,6 @@ inputs:
2118
runs:
2219
using: "composite"
2320
steps:
24-
- name: Checkout cts_exclude_filter folder
25-
uses: actions/checkout@v4
26-
with:
27-
ref: ${{ inputs.cts_exclude_ref }}
28-
path: cts_exclude
29-
sparse-checkout: |
30-
sycl/cts_exclude_filter
31-
- name: Move sycl to root
32-
shell: bash
33-
run: |
34-
mv cts_exclude/sycl .
35-
rm -rf cts_exclude
3621
- name: Checkout SYCL CTS tests
3722
if: inputs.cts_testing_mode != 'run-only'
3823
uses: ./devops/actions/cached_checkout

0 commit comments

Comments
 (0)