Skip to content

[CI] Set e2e test timeout to 1 hour #17748

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 31, 2025
Merged

[CI] Set e2e test timeout to 1 hour #17748

merged 4 commits into from
Mar 31, 2025

Conversation

sarnex
Copy link
Contributor

@sarnex sarnex commented Mar 31, 2025

We have it in LIT_OPTS already but we see sometimes it doesn't work, let's try this.

@sarnex sarnex requested a review from a team as a code owner March 31, 2025 15:20
Signed-off-by: Sarnie, Nick <[email protected]>
@sarnex sarnex temporarily deployed to WindowsCILock March 31, 2025 16:46 — with GitHub Actions Inactive
Signed-off-by: Sarnie, Nick <[email protected]>
@sarnex
Copy link
Contributor Author

sarnex commented Mar 31, 2025

@uditagarwal97 @aelovikov-intel Can you guys re-review, I had to change the PR because github doesn't support timeout-minutes in composite actions

https://github.com/orgs/community/discussions/50481

@aelovikov-intel
Copy link
Contributor

@uditagarwal97 @aelovikov-intel Can you guys re-review, I had to change the PR because github doesn't support timeout-minutes in composite actions

https://github.com/orgs/community/discussions/50481

GHA...

@@ -281,6 +281,7 @@ jobs:
- name: Run E2E Tests
if: inputs.tests_selector == 'e2e'
uses: ./devops/actions/run-tests/e2e
timeout-minutes: 65
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
timeout-minutes: 65
timeout-minutes: 60

Should be fine...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i did 65 because now the timeout includes checkout/pack/upload

Copy link
Contributor

Choose a reason for hiding this comment

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

I understand, but 60 was too generous to begin with. 55 should be enough.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok will change

Signed-off-by: Sarnie, Nick <[email protected]>
@sarnex sarnex merged commit d7c279b into intel:sycl Mar 31, 2025
17 of 25 checks passed
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