feat: payload size limit configurable

Author: yiweichi

crates/chain-orchestrator/src/metrics.rs

@@ -128,6 +128,6 @@ pub(crate) struct BlockBuildingMetric {
     pub(crate) gas_per_block: Histogram,
     /// The gas throughput in gas/second.
     pub(crate) gas_per_second: Histogram,
-    /// The cumulative gas used across all committed blocks (similar to geth's miner/commit_gas).
+    /// The cumulative gas used across all committed blocks (similar to geth's `miner/commit_gas`).
     pub(crate) commit_gas: Counter,
 }

crates/node/src/add_ons/mod.rs

@@ -1,7 +1,6 @@
 //! The [`ScrollRollupNodeAddOns`] implementation for the Scroll rollup node.
 
 use super::args::ScrollRollupNodeConfig;
-use crate::constants;
 
 use reth_evm::{ConfigureEngineEvm, EvmFactory, EvmFactoryFor};
 use reth_network::NetworkProtocols;
@@ -80,7 +79,7 @@ where
                 .with_min_suggested_priority_fee(
                     config.gas_price_oracle_args.default_suggested_priority_fee,
                 )
-                .with_payload_size_limit(constants::DEFAULT_PAYLOAD_SIZE_LIMIT)
+                .with_payload_size_limit(config.sequencer_args.payload_size_limit)
                 .with_sequencer(config.network_args.sequencer_url.clone()),
             ScrollEngineValidatorBuilder::default(),
             BasicEngineApiBuilder::default(),

crates/node/src/args.rs

@@ -735,6 +735,9 @@ pub struct SequencerArgs {
     /// The payload building duration for the sequencer (milliseconds)
     #[arg(long = "sequencer.payload-building-duration", id = "sequencer_payload_building_duration", value_name = "SEQUENCER_PAYLOAD_BUILDING_DURATION", default_value_t = constants::DEFAULT_PAYLOAD_BUILDING_DURATION)]
     pub payload_building_duration: u64,
+    /// The payload size limit for the sequencer (bytes)
+    #[arg(long = "sequencer.payload-size-limit", id = "sequencer_payload_size_limit", value_name = "SEQUENCER_PAYLOAD_SIZE_LIMIT", default_value_t = constants::DEFAULT_PAYLOAD_SIZE_LIMIT)]
+    pub payload_size_limit: u64,
     /// The fee recipient for the sequencer.
     #[arg(long = "sequencer.fee-recipient", id = "sequencer_fee_recipient", value_name = "SEQUENCER_FEE_RECIPIENT", default_value_t = SCROLL_FEE_VAULT_ADDRESS)]
     pub fee_recipient: Address,

crates/node/src/test_utils/mod.rs

@@ -78,7 +78,7 @@ pub use network_helpers::{
 use crate::{
     BlobProviderArgs, ChainOrchestratorArgs, ConsensusArgs, EngineDriverArgs, L1ProviderArgs,
     PprofArgs, RollupNodeDatabaseArgs, RollupNodeNetworkArgs, RpcArgs, ScrollRollupNode,
-    ScrollRollupNodeConfig, SequencerArgs,
+    ScrollRollupNodeConfig, SequencerArgs, constants,
 };
 use alloy_primitives::Bytes;
 use reth_chainspec::EthChainSpec;
@@ -274,6 +274,7 @@ pub fn default_sequencer_test_scroll_rollup_node_config() -> ScrollRollupNodeCon
             l1_message_inclusion_mode: L1MessageInclusionMode::BlockDepth(0),
             allow_empty_blocks: true,
             max_l1_messages: None,
+            payload_size_limit: constants::DEFAULT_PAYLOAD_SIZE_LIMIT,
         },
         blob_provider_args: BlobProviderArgs { mock: true, ..Default::default() },
         signer_args: Default::default(),