-
-
Notifications
You must be signed in to change notification settings - Fork 306
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: implement produce block v3 and move builder/execution race to beacon #5880
Conversation
3f85ce9
to
011d1bf
Compare
Performance Report✔️ no performance regression detected Full benchmark results
|
0481279
to
62d1e97
Compare
62d1e97
to
62d39f5
Compare
62d39f5
to
615bd59
Compare
3fc8b40
to
9ae9801
Compare
97aaf8e
to
25c4ab5
Compare
1e88930
to
e9c6aff
Compare
4779c67
to
9ea7037
Compare
@@ -77,45 +82,3 @@ export function processExecutionPayload( | |||
.getExecutionForkTypes(state.slot) | |||
.ExecutionPayloadHeader.toViewDU(payloadHeader) as typeof state.latestExecutionPayloadHeader; | |||
} | |||
|
|||
export function executionPayloadToPayloadHeader( |
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.
Note to self, util moved to packages/state-transition/src/util/execution.ts
Mostly cosmetic comments, otherwise looks good |
thank you will address the feedback ❤️ 🙏 |
Can you update the description with the lastest? |
88a2863
to
41abd3e
Compare
Is this really a breaking change if the flag is set to false by default? |
it is a lodestar cross version compatibility issue change as the RACE between execution and builder has now moved to beacon (for non produceblockv3 as well), so its both beacon and validator would need to be updated |
updated |
3daae60
to
e6573f0
Compare
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.
LGTM, thanks for fixing this PR up
9277614
to
acd897d
Compare
@dapplion further resolved the feedback 👍 ❤️ |
acd897d
to
676e315
Compare
update the impl interfaces Restore the previous versions update test fix oapi spec fix tests fix merge tests fix tests fix the tests implement produce blockv3 refac forktypes plug v3 into vlidator and get it working mock test produceblockv3 and fix issues fix lint fixes update head opts cleanup the validator flags shift fee recipient to extra args as well typo fix tests missed commit deprecate flag instead of removing backward produceblock v2 compatability lint disable produce blockv3 for mixed tests run on only execution sim option fixes improve log pass args fix spell check sort wordlist chore: sort wordlist chore: fix spell check refactor block types and introduce builder/execution race in produceblindedblock api as well fix the publish blinded block to pick locally cached blinded data add flag in get init dev vals fix unit test fixes switch to v2 as default fix issues and run sim tests on mix configurations remove utf8 graffiti from text log apply feedback on api cleanup on the sidecar cache blinded to full utils small produced cache cleanup add metrics cleanup response parsing lint further improv improve interface fix apply feedback
676e315
to
b643672
Compare
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.
LGTM!
🎉 This PR is included in v1.12.0 🎉 |
Breaking Change
(for annoucements/release notes)
race
has been moved to beacon (from validator) (via bothproduceBlockV3
andproduceBlindedBlock
) and hence both validation and beacon will need to be updatedvalidator flag--builder
is now deprecated and builder flow activation is solely decided upon--builder.selection
enabled
flag underbuilder
section. if you do not want to use the builder on a validator that might still be hooked onto a builder enabled beacon, you may provide the builderselection
asexecutiononly
(by default its set tomaxprofit
but default still can be overridden with--builder.selection
flag on the validator apart from the proposer config file level overrides)--builder.selection builderonly
flag on the validator to only trigger block production through the builder url configured on the beaconPart of
TODO:
Implements :
- [x] move produceblockv2 and produceblindedblock to the standard beacon apireverted after dicusssion in R&DPS:
this PR doesn't add consensus block value which will be provided in a followup PR