Skip to content

tests: extend timeout to 600 seconds#786

Merged
nvlsianpu merged 1 commit intonrfconnect:mainfrom
fundakol:fix-timeout-in-tests
May 4, 2026
Merged

tests: extend timeout to 600 seconds#786
nvlsianpu merged 1 commit intonrfconnect:mainfrom
fundakol:fix-timeout-in-tests

Conversation

@fundakol
Copy link
Copy Markdown
Contributor

Extended twister timeout for samples where additional boards has been added.

@fundakol fundakol requested review from a team as code owners April 30, 2026 11:07
@github-actions github-actions Bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Apr 30, 2026
@github-actions
Copy link
Copy Markdown

You can find the documentation preview for this PR here.

Extended twister timeout for samples where additional
boards has been added.

Signed-off-by: Lukasz Fundakowski <lukasz.fundakowski@nordicsemi.no>
@fundakol fundakol force-pushed the fix-timeout-in-tests branch from eaf4586 to 7ba698a Compare April 30, 2026 11:30
@fundakol fundakol changed the title tests: extend timeout top 600 seconds tests: extend timeout to 600 seconds Apr 30, 2026
Copy link
Copy Markdown
Contributor

@lemrey lemrey left a comment

Choose a reason for hiding this comment

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

Does that mean that these test take up to 10 minutes to execute each?

@fundakol
Copy link
Copy Markdown
Contributor Author

Does that mean that these test take up to 10 minutes to execute each?

Each scenario may contain a few tests written in pytest. 600 seconds timeout is for all those tests. A lower value is usually sufficient, but sometimes, under heavy load on the test node, the timeout is insufficient and the entire scenario ends with a timeout error. 600 seconds gives us more confidence.

@lemrey
Copy link
Copy Markdown
Contributor

lemrey commented Apr 30, 2026

Thanks for clarifying

@nvlsianpu nvlsianpu merged commit 712effa into nrfconnect:main May 4, 2026
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants