-
-
Notifications
You must be signed in to change notification settings - Fork 311
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.
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
feat: add logs for builder validator registrations #5986
Conversation
Performance Report✔️ no performance regression detected Full benchmark results
|
@@ -816,6 +816,10 @@ export function getValidatorApi({ | |||
}, | |||
|
|||
async registerValidator(registrations) { | |||
if (!chain.executionBuilder) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This API will now throw an error if a validator client tries to register validators and the beacon node does not have --builder
flag set. This is quite important in my opinion as current behavior is that this API just succeeds without actually registering validators with the builder. Without this check, the earliest we would detect a missing builder configuration on the beacon node would be when trying to produce a blinded block.
🎉 This PR is included in v1.12.0 🎉 |
Motivation
Closes #5337
Description
Adds logs for builder validator registrations. Especially on the validator client this should be logged to info as it helps to detect issues like rocket-pool/smartnode-install#105 earlier. On the beacon node side, I opted to keep it as debug for now and only log errors.
Note: For users that want more visibility on the beacon node, it is possible to set
--logLevelModule="api=debug"
flag. This will print out the debug logForwarded validator registrations ...
to stdout instead of just the log file.What are other clients doing?
INFO Published validator registrations to the builder network, count: 10, service: preparation
INFO Forwarding register validator request to connected builder, count: 9
INFO - Validator *** 10 out of 10 validator registration(s) were successfully sent to the builder network via the Beacon Node.
level=info msg="Submitted builder validator registration settings for custom builders" prefix=validator