From d97027fb43bf4398991a69b5073e5ddca27d41c4 Mon Sep 17 00:00:00 2001 From: tersec Date: Sun, 6 Aug 2023 15:12:32 +0000 Subject: [PATCH] remove less-than-useful warning (#5245) * remove less-than-useful warning * remove redundant check for node.config.payloadBuilderEnable --- beacon_chain/validators/validator_duties.nim | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/beacon_chain/validators/validator_duties.nim b/beacon_chain/validators/validator_duties.nim index 87e9dd3e3..3698a0065 100644 --- a/beacon_chain/validators/validator_duties.nim +++ b/beacon_chain/validators/validator_duties.nim @@ -809,14 +809,11 @@ proc proposeBlockAux( let payloadBuilderClientMaybe = node.getPayloadBuilderClient( validator_index.distinctBase) - if payloadBuilderClientMaybe.isErr: - warn "Unable to initialize payload builder client while proposing block", - err = payloadBuilderClientMaybe.error - else: + if payloadBuilderClientMaybe.isOk: payloadBuilderClient = payloadBuilderClientMaybe.get let usePayloadBuilder = - if node.config.payloadBuilderEnable and payloadBuilderClientMaybe.isOk: + if payloadBuilderClientMaybe.isOk: withState(node.dag.headState): # Head slot, not proposal slot, matters here # TODO it might make some sense to allow use of builder API if local