post-genesis transition #304
Closed
Conversation
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
This was referenced Jan 27, 2024
gballet
force-pushed
the
transition-post-genesis
branch
3 times, most recently
from
31f522b
to
009d3e2
Compare
…unt tree not a verkle tree (#321)
* add a switch to force proof in blocks * activate switch * fix switch type
* add switch to override the stride of the overlay conversion * set a default stride of 10k
* fix: save and load transition state for block processing * log whether the tree is verkle in LoadTransitionState * fix: ensure the transition is marked as started in insertChain * dump saved address * fix nil pointer panic * remove stacktrace that is no longer useful * fix a panic * fix build * check: copy current account address BEFORE it's saved * mandatory panic fix * Remove debug fmt.Println * more cleanup + comments
* cmd/{geth, utils}: add a command to clear verkle costs
fix: boolean issue
fix: load finalization state in FinalizeAndAssemble (#340)
* Conversion and TransitionTrie fixes (#346)
* fixes
Signed-off-by: Ignacio Hagopian <[email protected]>
* remove old comment
Signed-off-by: Ignacio Hagopian <[email protected]>
---------
Signed-off-by: Ignacio Hagopian <[email protected]>
* trace cleanup
---------
Signed-off-by: Ignacio Hagopian <[email protected]>
Co-authored-by: Ignacio Hagopian <[email protected]>
…n't happened. This is a workaround, because it will always be false when restarting a node that has started the transition but not necessarily completed it.
…tion states (#375) * persist conversion state to db * fix: don't recreate LRU when writing state * opt: only write state to db if not already present in LRU * fix: rlp can't encode TransitionState * fix: use gob because binary.Write does not support structs 🤦♂️ * fix: nil pointer panic * add logs to debug shadowfork * no such thing as not enough traces * ditto * fix stupid bug * add a comment for readability * add more traces * Lock the state transition during conversion (#384) * heavy handed approach: lock the state transition during conversion * also lock transition state loading/unloading * reduce logs verbosity * add conversion test to workflow (#386) * add conversion test to workflow * mandatory -f switch fix in rm * forgot & at the end of the geth command * remove incorrect kill * add debug traces * add an overlay stride * fix typo * Apply suggestions from code review
* eth: add debug_conversionStatus RPC call * add debug trace4s * Apply suggestions from code review * export started/ended fields
* fix post-verge sync * review: fix truncated comment
gballet
force-pushed
the
transition-post-genesis
branch
from
ec5b79e
to
bb3a39d
Compare
Merged
|
Converting to draft instead of closing, it is needed to distinguish at a glance to which "era" other PRs belong to. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Reactivate a post-genesis transition for a kaustinen-like testnet that goes through the conversion.
TransitionStatefor every block past the fork. It would be better if this system could be jettisoned after the conversion.kaustinen-with-shapella