-
-
Notifications
You must be signed in to change notification settings - Fork 478
feat: builder selection, broadcast validation, stateless flow for gloas block production #9595
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
Merged
Merged
Changes from 11 commits
Commits
Show all changes
55 commits
Select commit
Hold shift + click to select a range
b6f7c3a
feat: stateless flow for local block production post-gloas
nflaig ddcd146
inline envelope fork lookup
nflaig 374b535
rename flag to statelessBlockProduction and set default based on conf…
nflaig e8244d1
validate submitted blob data against bid commitments
nflaig 930d890
add Gloas to docs wordlist
nflaig 76a45a3
Merge branch 'unstable' into nflaig/stateless-flow
nflaig 50a8996
Merge remote-tracking branch 'origin/unstable' into nflaig/stateless-…
nflaig f65b099
require include_payload param in produceBlockV4
nflaig b447138
support builder selection params in produceBlockV4
nflaig ba362d6
rename statelessBlockProduction flag to payloadLocal
nflaig b65420b
harden broadcast validation when publishing execution payload envelope
nflaig 469a1a6
clarify why consensus checks always run for published envelopes
nflaig 8c67de6
align bid selection logs and censorship handling with produceBlockV3
nflaig 866b52f
add todo for envelope equivocation checks
nflaig 9eb96e0
clarify bid value payment is protocol-enforced
nflaig 03b46b6
simplify bid value comment
nflaig 025b68e
log suspected builder censorship as warn
nflaig 7b04cb9
fix stale comment on stateless flow default
nflaig 23f061b
harden execution payload envelope publishing edge cases
nflaig e7ac8f0
only report builder censorship if builder flow is enabled
nflaig f1b2b9f
clarify import signature verification comment
nflaig 7331786
clarify equivocation check todo
nflaig 39264ba
scope equivocation todo to envelope publishing
nflaig 0282a41
always treat envelope signature as verified on api import
nflaig 24eabeb
tweak comments
nflaig c0fdc24
Merge branch 'unstable' into nflaig/stateless-flow
nflaig 878e9f2
Merge branch 'unstable' into nflaig/stateless-flow
nflaig c0e5ad9
fix: only time data column sidecar computation when cells are computed
nflaig c03f295
return 404 when requesting envelope for a builder bid block
nflaig 9709fe1
Merge branch 'unstable' into nflaig/stateless-flow
nflaig 975693c
Apply suggestion from myself
nflaig b7b1b43
cover circuit breaker scenarios in unit tests
nflaig 6f2bd9d
treat builderonly as builderalways post-gloas
nflaig f5ea449
remove support for builderonly block selection
nflaig 58644c3
Merge branch 'unstable' into nflaig/stateless-flow
nflaig ddca7b2
warn and rewrite builderonly to builderalways on beacon node
nflaig 0474b09
clean up builder selection defaulting comment and test mocks
nflaig b2d14ad
forward builder selection post-gloas with fork-aware default
nflaig ac25374
Merge branch 'unstable' into nflaig/stateless-flow
nflaig ebebebb
rename signedEnvelope to signedEnvelopeOrContents
nflaig 97e5395
chore: add slot to missing cached block production error
nflaig 519af81
fix: return 500 for block state regeneration failure on envelope publish
nflaig c80042e
reuse single payload envelope input lookup on envelope publish
nflaig daa3c35
fix log
nflaig 7ffe483
include submitted blob count in envelope publish logs
nflaig 3390ca1
use blockRoot key in missing cached block production error
nflaig 51da9f7
track payload publish errors and enrich envelope publish error messages
nflaig a519000
include executionPayloadIncluded in published envelope log
nflaig 0312108
separate metrics
nflaig 8c69c54
lint
nflaig 09f35c8
improve gloas block proposal logs
nflaig fe599a2
Merge branch 'unstable' into nflaig/stateless-flow
nflaig 1093c43
re-add execution payload value to produceBlockV4
nflaig ef4f390
tweak comment
nflaig 21d1956
fix metrics
nflaig File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -45,6 +45,7 @@ Flamegraphs | |
| GPG | ||
| Geth | ||
| Github | ||
| Gloas | ||
| Goerli | ||
| Golang | ||
| Gossipsub | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.