Skip to content

Several tests timeout consistently under emulated aarch64 linux test suite #1801

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

Closed
jtattermusch opened this issue May 27, 2021 · 4 comments
Closed

Comments

@jtattermusch
Copy link
Contributor

Several tests (it seems it's almost always the same tests) report "Timeout of 2000ms exceeded". The problem could simply be that under emulator, the tests run more slowly which makes time timeout.

The tests:
js client -> js server Interop-adjacent tests max message size with a server with message size limits and a client without limits.should get an error when sending a large message
and
js client -> js server Interop-adjacent tests max message size with a server with message size limits and a client without limits.should get an error when sending a large message

There are also a few "after all" hook failures, not sure if they are related.

Example results:
https://source.cloud.google.com/results/invocations/91618033-4b5d-47f0-9d49-d7d42c61a414/targets/github%2Fgrpc-node%2Freports%2Fnode12/tests
https://source.cloud.google.com/results/invocations/fd64d2ec-62ac-4aad-ae79-1b4d414dce4b/targets/github%2Fgrpc-node%2Freports%2Fnode12/tests
https://source.cloud.google.com/results/invocations/ad72ca8e-c844-4047-bf99-0e3ee04d0c20/targets/github%2Fgrpc-node%2Freports%2Fnode12/tests
https://source.cloud.google.com/results/invocations/65a7687d-bb07-4fa1-8a5a-3aaea9aabc84/targets/github%2Fgrpc-node%2Freports%2Fnode12/tests

@jtattermusch
Copy link
Contributor Author

describe('Interop-adjacent tests', function() {

@jtattermusch
Copy link
Contributor Author

CC @murgatroid99 does this seem like a real problem or should we just increase the timeout of the test mentioned above?

@murgatroid99
Copy link
Member

I think we should just increase the timeouts. If they end up failing even with a generous timeout, then we can look for other issues. The errors in the hooks are probably nothing; sometimes asynchronous test errors get associated with the hook instead of with the test itself.

@jtattermusch
Copy link
Contributor Author

Fixed by #1803 (and its followups).

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

No branches or pull requests

2 participants