-
Notifications
You must be signed in to change notification settings - Fork 23
Closed
Labels
Description
Currently, if a panic occurs in integration tests on a background thread involved in sending server-side zmq messages, the thread gets poisoned silently and the test harness hangs because the client is waiting for zmq messages that can no longer be sent. This makes it hard to debug issues both locally and on CI.
To fix that, we could use the same approach as for the Ark binary where we propagate panics to the main thread and then panic from there:
Lines 245 to 249 in c6df0b2
| // This causes panics on background threads to propagate on the main | |
| // thread. If we don't propagate a background thread panic, the program | |
| // keeps running in an unstable state as all communications with this | |
| // thread will error out or panic. | |
| // https://stackoverflow.com/questions/35988775/how-can-i-cause-a-panic-on-a-thread-to-immediately-end-the-main-thread |