diff options
| author | Nathan Whitaker <17734409+nathanwhit@users.noreply.github.com> | 2024-03-29 23:23:48 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-03-29 23:23:48 -0700 |
| commit | 99720d0713ecc17245e9b08652aadcb160d92cb8 (patch) | |
| tree | 88682deabbc84b43c50ff058a6419f199e816cf2 /cli/mainrt.rs | |
| parent | 524e451bfbd36d3f5a00a16a7115b31b56805f10 (diff) | |
chore: Make jupyter integration tests less flaky and avoid hang (#23134)
There's a TOCTOU issue that can happen when selecting unused ports for
the server to use (we get assigned an unused port by the OS, and between
then and when the server actually binds to the port another test steals
it). Improve this by checking if the server existed soon after setup,
and if so we retry starting it. Client connection can also fail
spuriously (in local testing) so added a retry mechanism.
This also fixes a hang, where if the server exited (almost always due to
the issue described above) before we connected to it, attempting to
connect our client ZMQ sockets to it would just hang. To resolve this, I
added a timeout so we can't wait forever.
Diffstat (limited to 'cli/mainrt.rs')
0 files changed, 0 insertions, 0 deletions