Improve block lookup logging

[dapplion]

Issue Addressed

When dabbling in #5534, I noticed some code paths that should be covered by logging

Proposed Changes

Add some extra logs in block lookup

Additional Info

Noticed that SyncMessage::UnknownBlockHashFromAttestation only checks synced_and_connected whereas SyncMessage::UnknownParentBlob checks also is_execution_engine_online. Is this by design or a typo?

[realbigsean]

Noticed that SyncMessage::UnknownBlockHashFromAttestation only checks synced_and_connected whereas SyncMessage::UnknownParentBlob checks also is_execution_engine_online. Is this by design or a typo?

Looks like the synced method includes the EE check

    fn synced(&mut self) -> bool {
        self.network_globals().sync_state.read().is_synced()
            && self.network.is_execution_engine_online()
    }

[realbigsean]

Nice! Love logging improvements

[realbigsean]

@Mergifyio queue

[mergify]

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at b74da14