Skip to content

Ark: Panics in dummy kernel threads should be propagated to test harness #551

@lionel-

Description

@lionel-

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:

ark/crates/ark/src/main.rs

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

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions