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
Increase goroutinemap unit test timeouts. #25955
Conversation
50ms is flaky in Jenkins. Fixes kubernetes#25825
@swagiaal, you're the closest person who knows something about goroutinemaps, PTAL. |
1f6ea51
to
55f9148
Compare
After discussion in #25825, I increased the timeout to 1 minute and removed check that |
1 similar comment
LGTM |
buildcop says: this seems to make tests better and is low-risk, so marking p1. |
@k8s-bot test this [submit-queue is verifying that this PR is safe to merge] |
GCE e2e build/test passed for commit 55f9148. |
Automatic merge from submit-queue |
50ms is flaky in Jenkins. This makes the test to take at least 0.5s to check that things that should block and wait for something really do block and wait (was 100ms before).
Fixes #25825