-
Notifications
You must be signed in to change notification settings - Fork 20.6k
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
cmd/geth: implement dev mode for post-merge #27327
Merged
Merged
Changes from 1 commit
Commits
Show all changes
49 commits
Select commit
Hold shift + click to select a range
667a585
devmode (rebased again)
jwasinger b4d5583
break withdrawal logic into separate structure
jwasinger e5ca14a
fix/add comments
jwasinger 59850ce
rename clmock->simulated_beacon
jwasinger 797494a
remove unused file. remove unused code from node.go. move mutex ope…
jwasinger a33ad4a
move feeRecipient access into private function. remove some empty lines
jwasinger 3d7dc00
use shutdown channel instead of context and cancel function
jwasinger fe5db72
check if we have any transactions, without relying on the pending block
jwasinger c32fc8d
use Sign instead of Bitlen to check if big.Int == 0
jwasinger fa9267d
simulated_beacon: fixes
holiman 8d91eaa
don't validate withdrawal indices
jwasinger 5f04806
rename simulated_beacon package to simulatedbeacon. move under cmd/geth
jwasinger 650374a
lint
jwasinger 92441a1
cmd/geth/simulatedbeacon: refactorings
holiman 942a841
rename withdrawals->withdrawalQueue. re-include withdrawals in blocks
jwasinger a0938fb
remove usage of log.Crit
jwasinger 434cc20
fix withdrawals not being included when dev period is 0
jwasinger b7408af
don't register catalyst service twice in dev mode.
jwasinger c8c920d
rename simulatedbeacon files to reflect package name
jwasinger 59df562
Update cmd/geth/config.go
jwasinger 9c91d84
Update cmd/geth/simulatedbeacon/simulatedbeacon.go
jwasinger cca59ec
Update cmd/geth/simulatedbeacon/simulatedbeacon.go
jwasinger 06213ac
goimports
jwasinger 12b410a
add license
jwasinger 7d4d5cf
use common.Hash instead of *common.Hash for feeRecipient apis
jwasinger 30ca6f5
instantiate withdrawalQueue with new(...)
jwasinger f59982e
move withdrawals into single array in withdrawalQueue. rename getQue…
jwasinger 7c98200
remove uneccesary calls to engineAPI.ExchangeTransitionConfigurationV1
jwasinger f687ee4
move SimulatedBeacon.shutdownCh initialization into constructor
jwasinger 1bb92db
move simulatedbeacon package contents under eth/catalyst
jwasinger be5e29e
rename getFeeRecipient() -> feeRecipient
jwasinger c8d9ef3
Update eth/catalyst/simulated_beacon.go
jwasinger bb7b0cd
Update eth/catalyst/simulated_beacon.go
jwasinger 402ac2e
Update eth/catalyst/simulated_beacon.go
jwasinger 27f0bb0
move beginSealing and finalizeSealing out of loop
jwasinger 8d3330a
cmd/geth: prefer utils.Fatalf over returning an error object in makeF…
jwasinger e21dc29
only clear as many pending withdrawals from the queue as were include…
jwasinger e3ab2db
change dev.period flag to uint64 flag type
jwasinger 714bf98
don't register catalyst service if in light sync mode
jwasinger 4fd9197
better naming for feeRecipientMutex. move initialization of Simulate…
jwasinger 5eac860
Update eth/catalyst/simulated_beacon.go
jwasinger 1340352
wip: prepare for using new miner GetFullPayload api
jwasinger c6b50e9
eth/catalyst, miner: add GetFullPayload API
rjl493456442 e83d22d
somewhat works... but is mistakenly tracking multiple payloads simult…
jwasinger 68608ca
refactor simulated_beacon based on miner API changes
jwasinger 3a16d22
remove logging. add comment
jwasinger c592736
remove period from developer genesis configuration
jwasinger a504a80
comments
jwasinger ba286e5
cmd, eth/catalyst, miner: terminate block building by ResolveFull
rjl493456442 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
move SimulatedBeacon.shutdownCh initialization into constructor
- Loading branch information
commit f687ee4d2aefa711bd75d0d3504a13f0324738b4
There are no files selected for viewing
This file contains 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
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.
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.
You may want to use a waitGroup here to exit the loop elegantly.