Skip to content
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.

Sign up for GitHub

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

Jump to bottom

feat(op-reth): Support Snap Sync if Available Post-Bedrock #4209

Closed
1 task
Tracked by #8
, #4377
...
refcell opened this issue Aug 15, 2023 · 3 comments
Closed
1 task
Tracked by #8
, #4377
...

feat(op-reth): Support Snap Sync if Available Post-Bedrock #4209

refcell opened this issue Aug 15, 2023 · 3 comments
Labels
A-op-reth Related to Optimism and op-reth C-enhancement New feature or request S-stale This issue/PR is stale and will close with no further activity

Comments

@refcell
Copy link
Contributor

refcell commented Aug 15, 2023

Describe the feature

Diff Category: Node Modifications
Sub Category: Tx gossip disable option
op-geth source: eth/handler.go

Additional context

Overview

Snap sync is only supported post-bedrock. This issue is to allow snap sync for op-reth only after the configured bedrock block.

Where should this feature live?

Unknown until if and when snap sync is supported by reth.

Completion Criteria

  • Added check that chain block number is past bedrock under the optimism feature flag.

Resources
@refcell refcell added C-enhancement New feature or request S-needs-triage This issue needs to be labelled labels Aug 15, 2023
@refcell
Copy link
Contributor Author

refcell commented Aug 15, 2023

Request: A-op-reth label

@gakonst gakonst added the A-op-reth Related to Optimism and op-reth label Aug 15, 2023
@refcell refcell mentioned this issue Aug 16, 2023
Closed
@clabby clabby mentioned this issue Aug 28, 2023
Merged
@github-actions
Copy link
Contributor

github-actions bot commented Sep 1, 2023

This issue is stale because it has been open for 14 days with no activity.

@github-actions github-actions bot added the S-stale This issue/PR is stale and will close with no further activity label Sep 1, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Sep 9, 2023

This issue was closed because it has been inactive for 7 days since being marked as stale.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Sep 9, 2023
@github-project-automation github-project-automation bot moved this from Todo to Done in Reth Tracker Sep 9, 2023
@DaniPopes DaniPopes removed the S-needs-triage This issue needs to be labelled label Apr 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-op-reth Related to Optimism and op-reth C-enhancement New feature or request S-stale This issue/PR is stale and will close with no further activity
Projects
Reth Tracker
Archived in project
Milestone
No milestone
Development

No branches or pull requests
3 participants
@gakonst @refcell @DaniPopes