Skip to content

Conversation

@EwanC
Copy link
Contributor

@EwanC EwanC commented May 30, 2024

Several of the SYCL-Graph E2E tests occasionally fail in CI on Windows for unrelated PRs. We can replicate this locally with a lot of effort, but have not yet been able to diagnose the root cause of find a fix. Disable all the tests which have been documented as sporadically failing on Windows at any point.

See related GitHub Issues:

Several of the SYCL-Graph E2E tests occasionally fail in
CI for unrelated PRs. Disable all the tests which have
been documented as failing on Windows at any point.

See related GitHub Issues:

* intel#13951
* intel#12941
* intel#11852
@EwanC EwanC temporarily deployed to WindowsCILock May 30, 2024 09:39 — with GitHub Actions Inactive
@EwanC EwanC temporarily deployed to WindowsCILock May 30, 2024 09:53 — with GitHub Actions Inactive
@EwanC EwanC marked this pull request as ready for review May 30, 2024 11:26
@EwanC EwanC requested a review from a team as a code owner May 30, 2024 11:26
@EwanC
Copy link
Contributor Author

EwanC commented May 30, 2024

@intel/llvm-gatekeepers This is ready to merge

@sommerlukas sommerlukas merged commit 0a4f036 into intel:sycl May 30, 2024
EwanC pushed a commit to reble/llvm that referenced this pull request Jun 10, 2024
`test-e2e/Graph/Explicit/add_nodes_after_finalize.cpp` has
been reported as failing on an unrelated PR on Windows - intel#11852 (comment)

Disable this test in line with how other flaky graphs tests
have been disabled on Windows intel#13966

The `RecordReplay` equivalent of this Explicit test is already disabled
on Windows.
steffenlarsen pushed a commit that referenced this pull request Jun 10, 2024
`test-e2e/Graph/Explicit/add_nodes_after_finalize.cpp` has been reported
as failing on an unrelated PR on Windows -
#11852 (comment)

Disable this test in line with how other flaky graphs tests have been
disabled on Windows in #13966

The `RecordReplay` equivalent of this Explicit test is already disabled
on Windows.
ianayl pushed a commit to ianayl/sycl that referenced this pull request Jun 13, 2024
`test-e2e/Graph/Explicit/add_nodes_after_finalize.cpp` has been reported
as failing on an unrelated PR on Windows -
intel#11852 (comment)

Disable this test in line with how other flaky graphs tests have been
disabled on Windows in intel#13966

The `RecordReplay` equivalent of this Explicit test is already disabled
on Windows.
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