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.

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

op-batcher: control loop to throttle DA usage #12735

Merged
merged 4 commits into from
Nov 6, 2024
Merged

Conversation

roberto-bayardo
Copy link
Collaborator

@roberto-bayardo roberto-bayardo commented Oct 30, 2024

Description

This PR monitors batcher backlog and if it exceeds a threshold, has the execution engine build blocks with smaller DA requirements until the backlog is cleared.

Tests

e2e tests for both tx and block level da throttling.

Additional context

Metadata

@roberto-bayardo roberto-bayardo force-pushed the throttle-l1-data branch 3 times, most recently from 90bdfa1 to 63c096c Compare October 30, 2024 23:13
@roberto-bayardo roberto-bayardo force-pushed the throttle-l1-data branch 8 times, most recently from 843c33d to c23ca0c Compare November 4, 2024 16:43
op-batcher/batcher/driver.go Outdated Show resolved Hide resolved
op-batcher/batcher/config.go Outdated Show resolved Hide resolved
op-batcher/batcher/channel_manager.go Show resolved Hide resolved
op-batcher/batcher/driver.go Show resolved Hide resolved
op-batcher/flags/flags.go Show resolved Hide resolved
op-batcher/metrics/metrics.go Show resolved Hide resolved
op-e2e/interop/supersystem.go Outdated Show resolved Hide resolved
op-e2e/system/conductor/sequencer_failover_setup.go Outdated Show resolved Hide resolved
op-e2e/system/e2esys/setup.go Outdated Show resolved Hide resolved
@sebastianst sebastianst added this pull request to the merge queue Nov 6, 2024
@sebastianst sebastianst removed this pull request from the merge queue due to a manual request Nov 6, 2024
@sebastianst sebastianst added this pull request to the merge queue Nov 6, 2024
Merged via the queue into develop with commit dd3ec4c Nov 6, 2024
48 checks passed
@sebastianst sebastianst deleted the throttle-l1-data branch November 6, 2024 07:16
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