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.

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

main - spec alignment (v1.0.2): custom GasLimit , 1000 validators #1973 #1979

Merged
merged 13 commits into from
Jan 19, 2025

Conversation

y0sher
Copy link
Contributor

@y0sher y0sher commented Jan 13, 2025

Description

This PR updates and aligns to spec version of v1.0.2

  • It updates the test to support spec change of allowing bigger message size to support 1000 validators. (This change won't take effect until contract updates, so don't try)

  • Following the efforts of some Ethereum community members to increase the Gas Limit, we enable this feature in our SSV operator node.

Gas Limit

The SSV Operator node only uses gas limit when registering to MEV relay. It will be used for external blocks built using the consensus client with MEV. The execution node limits are set on the nodes themselves and will be used on local blocks. (check out https://pumpthegas.org/)

Setting it using :

ValidatorOptions:
    GasLimit: 35000000

or set the env var GAS_LIMIT to set the Gas Limit used for validator registrations. you should be able to check the validator registration gas fee in the relay using this endpoint:
<relay-url>/relay/v1/data/validator_registration?pubkey=<validator pk>

⚠️ Caution ⚠️

This update enables changing the gas limit for all the MEV proposals of the validators of the operator. ALL operators of the committee MUST set the same gas limit. Otherwise validator registrations will fail, changes to fee recipient might not take effect.

@y0sher y0sher requested a review from moshe-blox January 19, 2025 09:07
Copy link
Contributor

@moshe-blox moshe-blox left a comment

Choose a reason for hiding this comment

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

LGTM 🔥

@liorrutenberg liorrutenberg merged commit 7b9a7a0 into main Jan 19, 2025
11 of 12 checks passed
@y0sher y0sher mentioned this pull request Jan 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants