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: channel closed due to timeout didn't trigger posting data to the L1/DA layer #12827

Open
roberto-bayardo opened this issue Nov 5, 2024 · 0 comments · May be fixed by #12916
Open
Labels
A-op-batcher Area: op-batcher

Comments

@roberto-bayardo
Copy link
Collaborator

Bug Description

It's my understanding that when a channel times out, the data from the channel should start getting written to the L1 (or whatever the DA layer is). But I noticed a case where this didn't seem to happen:

The Base devnet has a max channel duration of 300 (== 1 hour) so it should be one hour at most between L1 writes. Despite this our alert fired the other day indicating no data had been posted for over 2 hours. When I looked into this, I see where the channel closed due to duration timeout, one hour since the previous write to L1. But I also see that it changed DA type to "calldata" here, requeued all the blocks, but then no data was posted to the L1 for another hour (when the next channel closed). Image below are the logs immediately following the first channel timeout.

Screenshot 2024-11-04 at 2 32 04 PM

Steps to Reproduce

Very hard to reproduce because you need to force the condition where the optimal DA type changes in between channel timeout.

Expected behavior

I expected that after the channel was closed due to timeout, the data would immediately start getting posted to the L1 instead of it taking another hour.

Environment Information:

Configurations:

Logs:

Additional context


⚠️ Notice: Issues that do not include the following sections will be subject to closure:

  • Bug Description
  • Steps to Reproduce
  • Environment Information

Please ensure all required sections are filled out accurately to expedite the debugging process and improve issue resolution efficiency.

@roberto-bayardo roberto-bayardo changed the title Channel Closed due to timeout didn't trigger posting data to the L1/DA layer op-batcher: channel closed due to timeout didn't trigger posting data to the L1/DA layer Nov 5, 2024
@geoknee geoknee added the A-op-batcher Area: op-batcher label Nov 12, 2024
@geoknee geoknee linked a pull request Nov 13, 2024 that will close this issue
@geoknee geoknee linked a pull request Nov 13, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-op-batcher Area: op-batcher
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants