lighthouse-pulse/validator_client
Paul Hauner 5e3fb13cfe Downgrade a CRIT in the VC for builder timeouts (#4366)
## Issue Addressed

NA

## Proposed Changes

Downgrade a `CRIT` to an `ERRO` when there's an `Irrecoverable` error whilst publishing a blinded block.

It's quite common for builders successfully broadcast a block to the network whilst failing to respond to the BN when it publishes a signed, blinded block. The VC is currently raising a `CRIT` when this happens and I think that's excessive.

These changes have the same intent as  #4073. In that PR I only managed to remove the `CRIT`s in the BN but missed this one in the VC.

I've also tidied the log messages to:

- Give them all the same title (*"Error whilst producing block"*) to help with grepping.
- Include the `block_slot` so it's easy to look up the slot in an explorer and see if it was actually skipped.

## Additional Info

This PR should not change any logic beyond logging.
2023-06-07 01:50:34 +00:00
..
slashing_protection Split common crates out into their own repos (#3890) 2023-04-28 01:15:40 +00:00
src Downgrade a CRIT in the VC for builder timeouts (#4366) 2023-06-07 01:50:34 +00:00
Cargo.toml Logging via the HTTP API (#4074) 2023-05-22 05:57:08 +00:00