Skip to content

[SYCL][E2E] Increase tests timeout#21523

Open
bratpiorka wants to merge 1 commit intointel:syclfrom
bratpiorka:rrudnick_bump_cuda_timeout
Open

[SYCL][E2E] Increase tests timeout#21523
bratpiorka wants to merge 1 commit intointel:syclfrom
bratpiorka:rrudnick_bump_cuda_timeout

Conversation

@bratpiorka
Copy link
Contributor

Increase tests timeout so these tests works on slower machines

@bratpiorka bratpiorka marked this pull request as ready for review March 16, 2026 11:08
@bratpiorka bratpiorka requested review from a team as code owners March 16, 2026 11:08
@bratpiorka
Copy link
Contributor Author

@intel/bindless-images-reviewers @intel/llvm-reviewers-runtime please review

@@ -1,3 +1,4 @@
// TIMEOUT: 600
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this documented anywhere? I didn't realize we can set individual test time limits like this and I don't see it used anywhere else.


if config.test_mode == "run-only":
lit_config.maxIndividualTestTime = 300
lit_config.maxIndividualTestTime = 1200
Copy link
Contributor

Choose a reason for hiding this comment

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

im not sure about this, can we not run with lower makeopts on this machines?

fyi @uditagarwal97

Copy link
Contributor

Choose a reason for hiding this comment

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

+1. If a test takes too long to run, we could instead try to split the test.

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