Skip to content

[CI][SYCL][NATIVE_CPU] Enable full e2e testing for Native CPU #20188

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Sep 25, 2025
Merged

[CI][SYCL][NATIVE_CPU] Enable full e2e testing for Native CPU #20188

merged 2 commits into from
Sep 25, 2025

Conversation

@coldav
Copy link

@coldav coldav commented Sep 24, 2025 •
edited
Loading

This enables running of e2e tests for the Native CPU job pre commit job.

Some tests have been xfailed or set to unsupported.

@coldav coldav requested a review from a team as a code owner September 24, 2025 14:02
sarnex
sarnex approved these changes Sep 24, 2025
View reviewed changes
Copy link
Contributor

@sarnex sarnex left a comment

Choose a reason for hiding this comment

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

change lgtm assuming tests pass, i would remove DRAFT: from the title tho

[CI][SYCL][NATIVE_CPU] Enable full e2e testing for Native CPU
9c07f80 
This enables running of e2e tests for the Native CPU job
@coldav coldav force-pushed the colin/enable_full_e2e_native_cpu branch from 1032508 to 9c07f80 Compare September 24, 2025 14:39
@coldav coldav requested review from a team as code owners September 24, 2025 17:20
@coldav coldav force-pushed the colin/enable_full_e2e_native_cpu branch from 2b62e42 to 7ed0bd1 Compare September 24, 2025 17:25
YuriPlyakhin
YuriPlyakhin approved these changes Sep 24, 2025
View reviewed changes
Copy link
Contributor

@YuriPlyakhin YuriPlyakhin left a comment

Choose a reason for hiding this comment

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

Same here. Assuming testing passes, LGTM

@coldav coldav force-pushed the colin/enable_full_e2e_native_cpu branch from 7ed0bd1 to ac2529f Compare September 24, 2025 18:51
@coldav coldav force-pushed the colin/enable_full_e2e_native_cpu branch from ac2529f to 41cac66 Compare September 24, 2025 20:09
@coldav coldav changed the title DRAFT: [CI][SYCL][NATIVE_CPU] Enable full e2e testing for Native CPU [CI][SYCL][NATIVE_CPU] Enable full e2e testing for Native CPU Sep 24, 2025
@github-actions
Copy link
Contributor

@intel/llvm-gatekeepers please consider merging

@coldav
Copy link
Author

coldav commented Sep 24, 2025

@YuriPlyakhin @sarnex . I've updated some xfails and unsupported since you would have looked at it.
@steffenlarsen this would add running of e2e tests with some extra xfails and unsupported. I've tried to add to the github issue to reflect it.

sarnex
sarnex approved these changes Sep 24, 2025
View reviewed changes
Copy link
Contributor

@sarnex sarnex left a comment

Choose a reason for hiding this comment

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

lgtm, note to other reviewers, the new full e2e testing is run on the build runners but it finishes in 2m 27s, so no flags from me

@sarnex sarnex requested a review from a team September 24, 2025 21:10
@steffenlarsen steffenlarsen merged commit 0832642 into intel:sycl Sep 25, 2025
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@YuriPlyakhin YuriPlyakhin YuriPlyakhin approved these changes

@sarnex sarnex sarnex approved these changes

@steffenlarsen steffenlarsen steffenlarsen approved these changes

@KseniyaTikhomirova KseniyaTikhomirova Awaiting requested review from KseniyaTikhomirova KseniyaTikhomirova is a code owner automatically assigned from intel/llvm-reviewers-runtime

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@coldav @YuriPlyakhin @sarnex @steffenlarsen