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

test/regress.h: Increase default timeval tolerance 50 ms -> 100 ms #1157

Merged
merged 1 commit into from Apr 26, 2021

Conversation

yifan19
Copy link

@yifan19 yifan19 commented Apr 23, 2021

The default timeout tolerance is 50 ms,
which causes intermittent failure in many the
related tests in arm64 QEMU.

See: https://bugzilla.yoctoproject.org/show_bug.cgi?id=14163
(The root cause seems to be a heavy load)

Signed-off-by: Yi Fan Yu yifan.yu@windriver.com

The default timeout tolerance is 50 ms,
which causes intermittent failure in many the
related tests in arm64 QEMU.

See: https://bugzilla.yoctoproject.org/show_bug.cgi?id=14163
(The root cause seems to be a heavy load)

Signed-off-by: Yi Fan Yu <yifan.yu@windriver.com>
@yifan19
Copy link
Author

yifan19 commented Apr 23, 2021

Perhaps there should be a flag/variable set so the tester can determine what is the appropriate tolerance in their testing environment?

@azat
Copy link
Member

azat commented Apr 24, 2021

Perhaps there should be a flag/variable set so the tester can determine what is the appropriate tolerance in their testing environment?

There is already such change for ratelim test, see 8a34869, but I haven't done this for regression tests, do you want do try?

@azat azat merged commit dff8fd2 into libevent:master Apr 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants