Skip to content

Commit 87a681e

Browse files
authored
[UR] Remove references to CTS_ADAPTER_PLATFORM in CI/docs (#18394)
This variable isn't actually read anywhere.
1 parent d9c5ddd commit 87a681e

File tree

3 files changed

+1
-30
lines changed

3 files changed

+1
-30
lines changed

.github/workflows/ur-build-hw.yml

Lines changed: 1 addition & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -13,11 +13,6 @@ on:
1313
runner_name:
1414
required: true
1515
type: string
16-
platform:
17-
description: "Platform string, `UR_CTS_ADAPTER_PLATFORM` will be set to this."
18-
required: false
19-
type: string
20-
default: ""
2116
static_loader:
2217
required: false
2318
type: string
@@ -46,11 +41,6 @@ on:
4641
runner_name:
4742
required: true
4843
type: string
49-
platform:
50-
description: "Platform string, `UR_CTS_ADAPTER_PLATFORM` will be set to this."
51-
required: false
52-
type: string
53-
default: ""
5444
static_loader:
5545
required: false
5646
type: string
@@ -89,7 +79,6 @@ jobs:
8979
{
9080
name: "${{inputs.adapter_name}}",
9181
other_name: "${{inputs.other_adapter_name}}",
92-
platform: "${{inputs.platform}}",
9382
static_Loader: "${{inputs.static_loader}}",
9483
static_adapter: "${{inputs.static_loader}}"
9584
}
@@ -105,7 +94,6 @@ jobs:
10594
steps:
10695
# TODO:
10796
# - investigate if DUR_CONFORMANCE_AMD_ARCH could be removed
108-
# - find better way to handle platform param (e.g. "Intel(R) OpenCL" -> "opencl")
10997
# - switch to Ninja generator in CMake
11098
# - downloading DPC++ should be integrated somehow; most likely use nightly release.
11199
#
@@ -184,7 +172,7 @@ jobs:
184172
- name: Test adapters
185173
env:
186174
ZE_ENABLE_LOADER_DEBUG_TRACE: 1
187-
run: env UR_CTS_ADAPTER_PLATFORM="${{matrix.adapter.platform}}" ctest -C ${{matrix.build_type}} --test-dir build --output-on-failure -L "conformance" --timeout 600 -VV
175+
run: ctest -C ${{matrix.build_type}} --test-dir build --output-on-failure -L "conformance" --timeout 600 -VV
188176

189177
- name: Get information about platform
190178
if: ${{ always() }}

.github/workflows/ur-precommit.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -77,12 +77,10 @@ jobs:
7777
image_options: -u 1001 --privileged --cap-add SYS_ADMIN --gpus all
7878
- name: OPENCL
7979
runner: UR_OPENCL
80-
platform: "Intel(R) OpenCL"
8180
docker_image: "ghcr.io/intel/llvm/ubuntu2204_build:latest"
8281
image_options: -u 1001 --device=/dev/dri --device=/dev/kfd --privileged --cap-add SYS_ADMIN
8382
- name: OPENCL
8483
runner: UR_OPENCL
85-
platform: "OPENCL:Intel(R) OpenCL"
8684
other_adapter: NATIVE_CPU
8785
docker_image: "ghcr.io/intel/llvm/ubuntu2204_build:latest"
8886
image_options: -u 1001 --device=/dev/dri --device=/dev/kfd --privileged --cap-add SYS_ADMIN
@@ -96,7 +94,6 @@ jobs:
9694
runner_name: ${{ matrix.runner }}
9795
static_loader: ${{ matrix.static || 'OFF' }}
9896
static_adapter: ${{ matrix.static || 'OFF' }}
99-
platform: ${{ matrix.platform || '' }}
10097
other_adapter_name: ${{ matrix.other_adapter || '' }}
10198
docker_image: ${{ matrix.docker_image || 'ghcr.io/intel/llvm/ubuntu2404_intel_drivers:alldeps'}}
10299
image_options: ${{ matrix.image_options || '' }}

unified-runtime/scripts/core/INTRO.rst

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -420,20 +420,6 @@ framework should run on. This can be used during development and testing to run
420420
are available. If both filters are specified, then they both must match a platform for it to be selected. If there are
421421
no valid platforms, then the tests will fail. Command line arguments take priority over these variables.
422422

423-
.. envvar:: UR_CTS_ADAPTER_PLATFORM
424-
425-
A specifier list in the form of `[(backend):](platform name)[;[(backend)]:(platform name)]...`. If a backend
426-
specific specifier is present in the list and the test is running for that backend, the device with the given name
427-
is chosen. Otherwise, it must match the name of the specifier from the list with no backend. Backend names are case-
428-
insensitive, however platform names must match exactly.
429-
430-
For example, if the test device has multiple platforms and you want to run tests on the "ABC Corp" backend when
431-
testing OpenCL and "XYZ Org" when testing level zero, you'd use `OPENCL:ABC Corp;LEVEL_ZERO:XYZ Org`. This form is
432-
useful when running the `build` target with a build with multiple backends.
433-
434-
For testing only one platform, the backend can be omitted. For example, just `ABC Corp` is sufficient if the tests
435-
are only going to be testing OpenCL.
436-
437423
.. envvar:: UR_CTS_BACKEND
438424

439425
A (case insensitive) backend to force the test to use. For example, `opencl`, `level_zero`, `hip` and so on.

0 commit comments

Comments
 (0)