Skip to content

Conversation

@KornevNikita
Copy link
Contributor

Disabled some redundant steps for run-only mode.
Test-run: https://github.com/intel/llvm/actions/runs/12708106400

Disabled some redundant steps for run-only mode.
@KornevNikita KornevNikita requested a review from a team as a code owner January 10, 2025 11:31
tests_selector: ${{ matrix.tests_selector }}
tests_selector: cts
ref: ${{ github.sha }}
merge_ref: ''
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This arg is not used in cts mode at all.

image: ghcr.io/intel/llvm/ubuntu2404_intel_drivers:latest
image_options: -u 1001 --device=/dev/dri --privileged --cap-add SYS_ADMIN
target_devices: opencl:cpu
tests_selector: cts
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i dont understand why we're removing this

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because this job's called run-sycl-cts, no e2e expected for this matrix.

Copy link
Contributor

@sarnex sarnex Jan 10, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

isn't this line telling it to run the cts tests instead of e2e?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh sorry, i see

Copy link
Contributor

@sarnex sarnex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

my bad i missed we were setting the selector all matrix entries to cts below

@aelovikov-intel aelovikov-intel merged commit 81c2858 into sycl Jan 10, 2025
24 checks passed
@bader bader deleted the update-nightly-cts branch January 11, 2025 01:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants