reverse_tunnels: Fix some flakes in the reverse connection cluster integration test #44065
Check was skipped
This check was not triggered in this CI run
Details
Request (pr/44065/main@8e12a38)
8e12a38 
#44065 merge main@e980367
Fix some flakes in the reverse connection cluster integration test
Commit Message
Fix some flakes in the reverse connection cluster integration test.
Additional Description
Wait for the listener to accept the connection before sending the requests to prevent a bunch of races that could come otherwise. We can take up finding and fixing the races later as it should not be easy to hit when run on different machines the accept window is usually very small only when done from a single machine does it become more likely.
Enable auto_close_connections by default othewise we risk letting both sockets be alive and reading from the same kernel socket which can cause other issues -> also add in ssl_quiet_close so the connection is not force closed by the ssl socket on the downstream side.
Testing
Unit and integ tests.
Environment
Request variables
| Key | Value |
|---|---|
| ref | 9803ec1 |
| sha | 8e12a38 |
| pr | 44065 |
| base-sha | e980367 |
| actor | @aakugan |
| message | Fix some flakes in the reverse connection cluster integration test... |
| started | 1774178744.133812 |
| target-branch | main |
| trusted | false |
Build image
Container image/s (as used in this CI run)
| Key | Value |
|---|---|
| default | docker.io/envoyproxy/envoy-build:86873047235e9b8232df989a5999b9bebf9db69c |
| mobile | docker.io/envoyproxy/envoy-build:mobile-86873047235e9b8232df989a5999b9bebf9db69c |
Version
Envoy version (as used in this CI run)
| Key | Value |
|---|---|
| major | 1 |
| minor | 38 |
| patch | 0 |
| dev | true |