Skip to content

Add variable builder deadline#28

Merged
ferranbt merged 4 commits intomainfrom
feat/arg-deadline
May 13, 2025
Merged

Add variable builder deadline#28
ferranbt merged 4 commits intomainfrom
feat/arg-deadline

Conversation

@ferranbt
Copy link
Contributor

📝 Summary

This PR introduces an argument to make the configure the extra deadline time to wait for the payload request before dropping the block building routine.

@ferranbt ferranbt requested review from SozinM and avalonche as code owners May 13, 2025 14:07
@ferranbt ferranbt mentioned this pull request May 13, 2025
3 tasks
#[arg(long = "builder.log-pool-transactions", default_value = "false")]
pub log_pool_transactions: bool,
/// How much time extra to wait for the block building job to complete and not get garbage collected
#[arg(long = "builder.extra-block-deadline-secs", default_value = "20")]
Copy link
Member

Choose a reason for hiding this comment

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

question: is it still --builder.extra-block-deadline-secs or --builder.extra-block-deadline-ms?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Seconds

@ferranbt ferranbt merged commit 907b06e into main May 13, 2025
3 checks passed
avalonche pushed a commit that referenced this pull request Jan 27, 2026
…g and add pre-exec validation (#341)

* Add external payload validation logic inside fb payload handler (#26)

* Remove post execution validation as default already does the validation (#27)

* Revert comments (#28)

* Fix execution txs logic on external payload validation (#31)

* Revert comment
mininny added a commit to testinprod-io/op-rbuilder that referenced this pull request Mar 5, 2026
…hbots#28)

* pin ci nighlty version 21-02

* Update circleci rust toolchain to nightly-2026-02-21

---------

Co-authored-by: julio4 <30329843+julio4@users.noreply.github.com>
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.

3 participants