Hitting unreachable in `DiscoveryManager`

[RCasatta]

This line is hit

At the moment it's not clear to me:

• why the entry is supposed to be always found and the reason of unreachable
• why electrs crash since the panic should be inside a spawned thread

[philippem]

@shesek any idea?

[shesek]

why the entry is supposed to be always found

It should be impossible to have a job in the queue that was healthy on the last check without a corresponding entry in the healthy set. I went over the code again and am still unsure how this could happen.

Are you hitting this consistently? Any reproduction steps?

and the reason of unreachable

I generally prefer to use unreachables for impossible/inconsistent states, but changing this to a warn would be ok too.

why electrs crash since the panic should be inside a spawned thread

I'm not sure, looking into this.