Skip to content

[CI] Add Windows BMG testing in precommit #17105

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Mar 6, 2025
Merged

[CI] Add Windows BMG testing in precommit #17105

merged 4 commits into from
Mar 6, 2025

Conversation

sarnex
Copy link
Contributor

@sarnex sarnex commented Feb 20, 2025

We got a Windows BMG machine, so add Windows BMG testing.

There are two categories of failures here: consistent failures/timeouts and really slow tests causing CI to take 30min extra.

I disabled both of these and made two GH issues: one, two.

@sarnex sarnex closed this Feb 25, 2025
@sarnex sarnex reopened this Feb 25, 2025
@sarnex sarnex temporarily deployed to WindowsCILock March 5, 2025 20:19 — with GitHub Actions Inactive
@sarnex sarnex temporarily deployed to WindowsCILock March 5, 2025 20:36 — with GitHub Actions Inactive
@sarnex sarnex temporarily deployed to WindowsCILock March 5, 2025 20:37 — with GitHub Actions Inactive
@sarnex sarnex marked this pull request as ready for review March 5, 2025 21:21
@sarnex sarnex requested review from a team as code owners March 5, 2025 21:21
@sarnex sarnex requested review from EwanC and againull March 5, 2025 21:22
Copy link
Contributor

@dkhaldi dkhaldi left a comment

Choose a reason for hiding this comment

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

Thank you for adding this.

Signed-off-by: Sarnie, Nick <[email protected]>
@sarnex sarnex temporarily deployed to WindowsCILock March 6, 2025 15:21 — with GitHub Actions Inactive
sarnex added 2 commits March 6, 2025 10:39
Signed-off-by: Sarnie, Nick <[email protected]>
Signed-off-by: Sarnie, Nick <[email protected]>
@sarnex sarnex temporarily deployed to WindowsCILock March 6, 2025 18:41 — with GitHub Actions Inactive
@sarnex sarnex temporarily deployed to WindowsCILock March 6, 2025 20:59 — with GitHub Actions Inactive
@sarnex sarnex temporarily deployed to WindowsCILock March 6, 2025 21:00 — with GitHub Actions Inactive
@sarnex sarnex merged commit 2054243 into intel:sycl Mar 6, 2025
23 checks passed
AlexeySachkov pushed a commit to AlexeySachkov/llvm that referenced this pull request Mar 31, 2025
We got a Windows BMG machine, so add Windows BMG testing.

There are two categories of failures here: consistent failures/timeouts
and really slow tests causing CI to take 30min extra.

I disabled both of these and made two GH issues:
[one](intel#17165),
[two](intel#17255).

---------

Signed-off-by: Sarnie, Nick <[email protected]>
AlexeySachkov added a commit that referenced this pull request Apr 1, 2025
This is a joined cherry-pick of #17347 and #17105
made to align disabled tests between `sycl` and `sycl-rel-6_1_0`
branches to make the pre-commit more stable on the latter.

Descriptions of cherry-picked commits:

---

[SYCL][Graph][E2E] Disable all Graph E2E testing on Windows Battlemage
(#17348)

We currently only disable the RecordReplay variant of E2E testing on
battlemage, however we have many other tests and the likihood is that if
the RecordReplay tests are failing then other tests will start failing
too.

Disable all Graphs tests until Graph support on Windows battlemage can
be investigated more thoroughly.

See #17165

Patch-By: Ewan Crawford <[email protected]>

---

[CI] Add Windows BMG testing in precommit (#17105)

We got a Windows BMG machine, so add Windows BMG testing.

There are two categories of failures here: consistent failures/timeouts
and really slow tests causing CI to take 30min extra.

I disabled both of these and made two GH issues:
[one](#17165),
[two](#17255).

Patch-by: Sarnie, Nick <[email protected]>

---
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.

6 participants