Skip to content

[CI] Setup vulkan env for building e2e tests#21247

Merged
sarnex merged 4 commits intointel:syclfrom
sarnex:civulkan
Feb 10, 2026
Merged

[CI] Setup vulkan env for building e2e tests#21247
sarnex merged 4 commits intointel:syclfrom
sarnex:civulkan

Conversation

@sarnex
Copy link
Contributor

@sarnex sarnex commented Feb 9, 2026

Two problems:

  1. The vulkan env wasn't set up when building e2e tests so those tests weren't build, but at test-time the env was set up so they ran, but the test wasn't build so you get a test_exe_not_found error

  2. After 1) is fixed, my fix for Vulkan on NVIDIA had a problem, need to install one extra package. Manually verified this fixes it.

  3. A hidden existing issue, on 24.04 the vulkan script wasn't running because apt was erroring about some clang-format dependency problem (we need to use clang-format-22 but we only have the unversioned apt repo entry which has moved on to 23, so add 22 explicitly) but the script didn't error because we didn't have set -x. Fix the dependency problem and the script to error on failure.

Closes: #21239

Signed-off-by: Nick Sarnie <nick.sarnie@intel.com>
@ldorau
Copy link
Contributor

ldorau commented Feb 9, 2026

@sarnex Thanks for the patch! I am testing it in #21110 and #21242

@sarnex
Copy link
Contributor Author

sarnex commented Feb 9, 2026

Sure, hopefully it works :)

Signed-off-by: Nick Sarnie <nick.sarnie@intel.com>
@sarnex sarnex marked this pull request as ready for review February 9, 2026 22:23
@sarnex sarnex requested a review from a team as a code owner February 9, 2026 22:23
@sarnex sarnex requested a review from uditagarwal97 February 9, 2026 22:23
Signed-off-by: Nick Sarnie <nick.sarnie@intel.com>
@sarnex
Copy link
Contributor Author

sarnex commented Feb 9, 2026

@uditagarwal97 Just pushed a new commit fixing a new problem I found, described in issue 3 in the description, hopefully it looks okay.

Signed-off-by: Nick Sarnie <nick.sarnie@intel.com>
@uditagarwal97
Copy link
Contributor

@uditagarwal97 Just pushed a new commit fixing a new problem I found, described in issue 3 in the description, hopefully it looks okay.

The new commit looks good.

@ldorau
Copy link
Contributor

ldorau commented Feb 10, 2026

@sarnex
Copy link
Contributor Author

sarnex commented Feb 10, 2026

@ldorau That's expected, the new container isn't being used, we need to merge this PR first and then wait for the containers to be built+published.

@sarnex sarnex merged commit bc1d138 into intel:sycl Feb 10, 2026
49 of 54 checks passed
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.

Vulkan E2E tests fail with the "command not found" error

3 participants

Comments