-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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/integration:api_listener_integration_test flakes #9746
Labels
Comments
Thanks for reporting @lizan! I'll look at it today |
lizan
pushed a commit
that referenced
this issue
Jan 24, 2020
Description: previously the test was not waiting for the expectation on the server's thread to complete. Therefore, there was a use after free race condition with the ApiListener's TlsCachingDateProvider. This PR makes it so that the test waits for the expectation to be fulfilled and thus prevents the race. Risk Level: low Testing: ran integration test a few thousand times locally on a linux machine. Fixes #9746 Signed-off-by: Jose Nino <[email protected]>
junr03
added a commit
that referenced
this issue
Feb 8, 2020
…9959) Description: this PR adds a shutdown method to the ApiListener and calls it where appropriate during server termination. Previously there would be a crash due to use after free of objects in thread local storage by streams in the ApiListener. Funny enough the flakes reported in #9746 happened due to this. Risk Level: low Testing: new unit and integration test. Without appropriate termination the new integration test repros the stacktrace reported in #9746. Signed-off-by: Jose Nino <[email protected]>
antoniovicente
pushed a commit
to antoniovicente/envoy
that referenced
this issue
Sep 29, 2020
Description: previously the test was not waiting for the expectation on the server's thread to complete. Therefore, there was a use after free race condition with the ApiListener's TlsCachingDateProvider. This PR makes it so that the test waits for the expectation to be fulfilled and thus prevents the race. Risk Level: low Testing: ran integration test a few thousand times locally on a linux machine. Fixes envoyproxy#9746 Signed-off-by: Jose Nino <[email protected]> Signed-off-by: Antonio Vicente <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
bazel test //test/integration:api_listener_integration_test --runs_per_test=1000
results:Call Stack:
The text was updated successfully, but these errors were encountered: