Skip to content

[SYCL][E2E] Disable failing L0 tests on Win DG2#22630

Merged
sarnex merged 2 commits into
intel:syclfrom
sarnex:windg2fix2
Jul 17, 2026
Merged

[SYCL][E2E] Disable failing L0 tests on Win DG2#22630
sarnex merged 2 commits into
intel:syclfrom
sarnex:windg2fix2

Conversation

@sarnex

@sarnex sarnex commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Sporadically failing, see #22033, #21556 and #21766

sarnex added 2 commits July 14, 2026 11:37
Signed-off-by: Nick Sarnie <nick.sarnie@intel.com>
Signed-off-by: Nick Sarnie <nick.sarnie@intel.com>
@sarnex sarnex changed the title [SYCL][E2E] Disable failing L0 adapter tests on Win DG2 [SYCL][E2E] Disable failing L0 tests on Win DG2 Jul 14, 2026
@sarnex
sarnex marked this pull request as ready for review July 14, 2026 20:13
@sarnex
sarnex requested review from a team as code owners July 14, 2026 20:13
@sarnex
sarnex requested a review from dm-vodopyanov July 14, 2026 20:13
@dm-vodopyanov

Copy link
Copy Markdown
Contributor

Should // UNSUPPORTED-TRACKER: be updated as well?

@sarnex

sarnex commented Jul 15, 2026

Copy link
Copy Markdown
Contributor Author

Edit: Ignore this, this is wrong

I don't think so, these tests fundamentally aren't supported on DG2, they never worked and will never work because they use hardware features that aren't available on DG2.

I think we just made a mistake adding this test originally where we mistakenly thought it would work on DG2.

dm-vodopyanov
dm-vodopyanov previously approved these changes Jul 15, 2026
@sarnex

sarnex commented Jul 15, 2026

Copy link
Copy Markdown
Contributor Author

@dm-vodopyanov Sorry, I am totally wrong, I thought this was a review for another PR
I updated each listed UNSUPPORTED-TRACKER to contain the DG2 failure and I think it's probably the same root cause, so IMO we don't need new trackers.

@sarnex
sarnex dismissed dm-vodopyanov’s stale review July 15, 2026 14:27

i gave wrong info

@sarnex

sarnex commented Jul 15, 2026

Copy link
Copy Markdown
Contributor Author

@dm-vodopyanov Could you take a second look at this pls? Thx

@sarnex
sarnex merged commit 97eccf7 into intel:sycl Jul 17, 2026
47 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.

2 participants