Include worker PID in ProcessExpired exception

[rsiemens]

👋 , we are utilizing the pebble ProcessPool quite heavily and occasionally we'll find ourselves needing to debug an abnormal exit on a worker. When ProcessExpired is raised in the main processes done_callback we get the exit code, which is helpful, but lack the process id of the worker which would make finding logs and sourcing the underlying reason much easier.

Ask:

• Can we update handle_worker_expiration to include the worker_id in the ProcessExpired exception here?

[noxdafox]

Hello,

thanks for reporting this issue. I will add the PID onto the exception.

[charettes]

Thanks for your consideration @noxdafox 🙇 Would submitting a PR with tests tweaks ourselves be appreciated or you'd prefer to tackle the feature request yourself?

I see there's already a test we could adjust for this purpose