Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fee-recipient: update error log to warn log #10684

Merged
merged 7 commits into from
May 16, 2022
Merged

fee-recipient: update error log to warn log #10684

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
df56ab9
initial commit
james-prysm May 12, 2022
d5bafa4
updating fee recipient on beacon node
james-prysm May 12, 2022
b1f1712
Merge branch 'develop' into fee-recipient-log-update
rauljordan May 13, 2022
5904cb3
Merge branch 'develop' into fee-recipient-log-update
james-prysm May 13, 2022
f1a0ef1
updating logs
james-prysm May 13, 2022
08700f3
Merge branch 'develop' into fee-recipient-log-update
james-prysm May 13, 2022
c3065d2
Merge branch 'develop' into fee-recipient-log-update
james-prysm May 16, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion beacon-chain/blockchain/execution_engine.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -282,7 +282,9 @@ func (s *Service) getPayloadAttribute(ctx context.Context, st state.BeaconState,
logrus.WithFields(logrus.Fields{
"validatorIndex": proposerID,
"burnAddress": fieldparams.EthBurnAddressHex,
}).Error("Fee recipient not set. Using burn address")
}).Warn("Fee recipient not set. Using burn address. " +
"You will not be rewarded transaction fees on this setting. " +
"Please set a different fee recipient.")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's avoid periods in our logs. Also, we should perhaps tell them to read our documentation regarding fee recipients?

}
case err != nil:
return false, nil, 0, errors.Wrap(err, "could not get fee recipient in db")
Expand Down
4 changes: 3 additions & 1 deletion beacon-chain/rpc/prysm/v1alpha1/validator/proposer_execution_payload.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -132,7 +132,9 @@ func (vs *Server) getExecutionPayload(ctx context.Context, slot types.Slot, vIdx
logrus.WithFields(logrus.Fields{
"validatorIndex": vIdx,
"burnAddress": common.BytesToAddress(burnAddr).Hex(),
}).Error("Fee recipient not set. Using burn address")
}).Warn("Fee recipient not set. Using burn address." +
"You will not be rewarded transaction fees on this setting. " +
"Please set a different fee recipient.")
}
default:
return nil, errors.Wrap(err, "could not get fee recipient in db")
Expand Down