Skip to content

[CI] Add LTS driver job to nightly - #22935

Merged
sarnex merged 1 commit into
syclfrom
sarnex/nightly
Aug 13, 2026
Merged

[CI] Add LTS driver job to nightly#22935
sarnex merged 1 commit into
syclfrom
sarnex/nightly

Conversation

@sarnex

@sarnex sarnex commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Use the new LTS docker container in the nightly to lock down we didn't completely break support on the LTS driver, which we need to support.

Note this runs on a runner with a special label, pvc_lts, as we also need the LTS kernel driver for true verification. This runner is only used for this job (I took a machine which had 2 PVC cards and put 1 of them into a VM with the LTS kernel driver, the existing runner for this machine only used 1 PVC card anyway, so there is no slowdown)

A major difference with normal E2E jobs is we don't run all/most tests, actually most tests fail. I think (hope?) this is expected because it's fine if we require a new driver for new SYCL features and probably most tests are for new features, the main idea is to lock down we didn't universally enable some new SPIR-V extension or something that completely breaks everything.

I asked an LLM to pick 5 tests to maximize coverage out of the ~270 passing tests, and made a new file in devops that lists them. This file will be read by the job and end up as the LIT_FILTER envvar.

I didn't want to enable/disable tests directly in E2E code as we'd have to either mark thousands of failing tests as UNSUPPORTED or make this config UNSUPPORTED by default and remove UNSUPPORTED from hundreds of passing tests using lit.local.cfg, seems much more trouble than it's worth.

I went with a separate file in devops instead hardcoding it in the job for better extendability and understandability.

Example CI run: https://github.com/intel/llvm/actions/runs/31633290719/job/94240857548

Comment thread .github/workflows/sycl-nightly.yml Fixed
Signed-off-by: Nick Sarnie <nick.sarnie@intel.com>

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@sarnex
sarnex marked this pull request as ready for review August 12, 2026 21:04
@sarnex
sarnex requested a review from a team as a code owner August 12, 2026 21:04
@sarnex
sarnex merged commit 21097c6 into sycl Aug 13, 2026
31 of 32 checks passed
@sarnex
sarnex deleted the sarnex/nightly branch August 13, 2026 14:32
@sarnex

sarnex commented Aug 13, 2026

Copy link
Copy Markdown
Contributor Author

thanks!

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.

3 participants