Skip to content

chore(pectra): Add Pectra Fork Logging (3/n) - #2637

Merged
rezzmah merged 239 commits into
mainfrom
timestamp-based-forking-devnet-chainspec-1
Apr 4, 2025
Merged

chore(pectra): Add Pectra Fork Logging (3/n)#2637
rezzmah merged 239 commits into
mainfrom
timestamp-based-forking-devnet-chainspec-1

Conversation

@rezzmah

@rezzmah rezzmah commented Mar 29, 2025

Copy link
Copy Markdown
Contributor

Follows #2636

Adds logging and reth flags to local dev

@rezzmah rezzmah changed the title pocing [ignore] chore(pectra): Add Pectra Fork Logging Apr 1, 2025
@rezzmah rezzmah changed the title chore(pectra): Add Pectra Fork Logging chore(pectra): Add Pectra Fork Logging (3/n) Apr 1, 2025
Comment thread scripts/build/testing.mk
Comment on lines +82 to +84
--ipcpath ${IPC_PATH} \
--engine.persistence-threshold 0 \
--engine.memory-block-buffer-target 0

@rezzmah rezzmah Apr 1, 2025

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Reth needs these flags or it borks due to not persisting blocks. Adding to local.

Could also add to testnet/mainnet

@rezzmah
rezzmah marked this pull request as ready for review April 1, 2025 20:58
@rezzmah
rezzmah requested a review from a team as a code owner April 1, 2025 20:58
Base automatically changed from timestamp-based-forking to main April 3, 2025 11:40
Signed-off-by: Shota Ehrlich <shotaehrlich@yahoo.com>

@shotes shotes left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@rezzmah
rezzmah merged commit 090d0b7 into main Apr 4, 2025
@rezzmah
rezzmah deleted the timestamp-based-forking-devnet-chainspec-1 branch April 4, 2025 00:42
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.

4 participants