Skip to content

add timeout scaling with new metadata 'timeout_scale'#280

Open
svmhdvn wants to merge 1 commit into
freebsd:masterfrom
svmhdvn:push-nsklrmttomxq
Open

add timeout scaling with new metadata 'timeout_scale'#280
svmhdvn wants to merge 1 commit into
freebsd:masterfrom
svmhdvn:push-nsklrmttomxq

Conversation

@svmhdvn

@svmhdvn svmhdvn commented May 21, 2026

Copy link
Copy Markdown
Contributor

As noted in the manpage, this can be used to account for differences in performance characteristics between execution environments at a global level.

For example, if tests on average take < 5 minutes (current default timeout of 300s) on native x86_64, but take < 20 mins on QEMU-emulated aarch64, timeout_scale = 4 can be set in kyua.conf on the emulated platform only. This can also be narrowly set on a per-test-suite basis to account for a group of exceedingly-long test cases.

@ihoro ihoro left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@ihoro

ihoro commented May 24, 2026

Copy link
Copy Markdown
Member

Perhaps, we want to fix the missing Kyuafile by a separate commit/PR.

@svmhdvn svmhdvn force-pushed the push-nsklrmttomxq branch from f6b379f to 90ebb63 Compare May 25, 2026 15:03
@svmhdvn

svmhdvn commented May 25, 2026

Copy link
Copy Markdown
Contributor Author

Perhaps, we want to fix the missing Kyuafile by a separate commit/PR.

Thanks, moved that to #281

Signed-off-by:	Siva Mahadevan <siva@FreeBSD.org>
@svmhdvn svmhdvn force-pushed the push-nsklrmttomxq branch from 90ebb63 to 941aede Compare May 29, 2026 17:34
@svmhdvn

svmhdvn commented May 29, 2026

Copy link
Copy Markdown
Contributor Author

Added Signed-off-by tag in the latest commit. And here are the full test results with this change:

kyua_test_results.txt

@ihoro

ihoro commented May 29, 2026

Copy link
Copy Markdown
Member

It's time to ask Enji to take a look.

@ihoro ihoro requested a review from ngie-eign May 29, 2026 19:50
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.

2 participants