Skip to content
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

[CI][CTS] Add filter for build-only mode #16671

Merged
merged 3 commits into from
Jan 20, 2025
Merged

Conversation

KornevNikita
Copy link
Contributor

In nightly we build and run CTS separately, so the build contains the full set of tests. Adding a filter to exclude categories that are not supported at all (e.g. compfail).

In nightly we build and run CTS separately, so the build contains the
full set of tests. Adding a filter to exclude categories that are not
supported at all (e.g. compfail).
Copy link
Contributor

@aelovikov-intel aelovikov-intel left a comment

Choose a reason for hiding this comment

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

I'd prefer cts_known_compfails, but that's subjective.

@@ -0,0 +1,2 @@
# Please use "#" to add comments here.
Copy link
Contributor

Choose a reason for hiding this comment

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

this will be used in the future to filter compfails, but is doing nothing today, right?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Exactly

@KornevNikita
Copy link
Contributor Author

Basic/fpga_tests/fpga_pipes.cpp - #16693
SubGroup/load_store.cpp - #16694
bindless_images/3_channel_format.cpp - #16691
Adapters/level_zero_batch_event_status.cpp - #16695

@intel/llvm-gatekeepers could you please merge?

@dm-vodopyanov dm-vodopyanov merged commit 38e6e1b into sycl Jan 20, 2025
23 of 26 checks passed
@bader bader deleted the cts-compile-filter branch January 23, 2025 04:46
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.

4 participants