Skip to content

Commit 67046c8

Browse files
chore(apollo_integration_tests): override lag_margin_seconds for gas price provider (#6071)
1 parent 1a924c3 commit 67046c8

File tree

4 files changed

+9
-14
lines changed

4 files changed

+9
-14
lines changed

config/sequencer/presets/system_test_presets/single_node/node_0/executable_0/node_config.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -217,7 +217,7 @@
217217
"gateway_config.stateless_tx_validator_config.validate_non_zero_resource_bounds": true,
218218
"http_server_config.ip": "127.0.0.1",
219219
"http_server_config.port": 23000,
220-
"l1_gas_price_provider_config.lag_margin_seconds": 60,
220+
"l1_gas_price_provider_config.lag_margin_seconds": 0,
221221
"l1_gas_price_provider_config.number_of_blocks_for_mean": 300,
222222
"l1_gas_price_provider_config.storage_limit": 3000,
223223
"l1_gas_price_scraper_config.finality": 0,

crates/apollo_integration_tests/src/flow_test_setup.rs

-1
Original file line numberDiff line numberDiff line change
@@ -261,7 +261,6 @@ impl FlowSequencerSetup {
261261
let num_l1_txs = u64::try_from(NUM_L1_TRANSACTIONS).unwrap();
262262
node_config.l1_gas_price_scraper_config.number_of_blocks_for_mean = num_l1_txs;
263263
node_config.l1_gas_price_provider_config.number_of_blocks_for_mean = num_l1_txs;
264-
node_config.l1_gas_price_provider_config.lag_margin_seconds = 0;
265264

266265
debug!("Sequencer config: {:#?}", node_config);
267266
let (clients, servers) = create_node_modules(&node_config).await;

crates/apollo_integration_tests/src/integration_test_manager.rs

+1-12
Original file line numberDiff line numberDiff line change
@@ -259,7 +259,7 @@ impl IntegrationTestManager {
259259
) -> Self {
260260
let tx_generator = create_integration_test_tx_generator();
261261

262-
let (mut sequencers_setup, node_indices) = get_sequencer_setup_configs(
262+
let (sequencers_setup, node_indices) = get_sequencer_setup_configs(
263263
&tx_generator,
264264
num_of_consolidated_nodes,
265265
num_of_distributed_nodes,
@@ -283,17 +283,6 @@ impl IntegrationTestManager {
283283
make_block_history_on_anvil(&anvil, base_layer_config.clone(), num_blocks_needed_on_l1)
284284
.await;
285285

286-
for seq in sequencers_setup.iter_mut() {
287-
for exec in seq.executables.iter_mut() {
288-
// Set the L1 gas price scraper's lag time to zero, to be able to use the
289-
// anvil transactions we send in make_block_history_on_anvil.
290-
exec.base_app_config.modify_config(|config| {
291-
config.l1_gas_price_provider_config.lag_margin_seconds = 0
292-
});
293-
exec.base_app_config.dump_config_file(&exec.node_config_path);
294-
}
295-
}
296-
297286
let idle_nodes = create_map(sequencers_setup, |node| node.get_node_index());
298287
let running_nodes = HashMap::new();
299288

crates/apollo_integration_tests/src/utils.rs

+7
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@ use apollo_gateway::config::{
2323
use apollo_http_server::test_utils::create_http_server_config;
2424
use apollo_infra_utils::test_utils::AvailablePorts;
2525
use apollo_l1_gas_price::eth_to_strk_oracle::{EthToStrkOracleConfig, ETH_TO_STRK_QUANTIZATION};
26+
use apollo_l1_gas_price::l1_gas_price_provider::L1GasPriceProviderConfig;
2627
use apollo_l1_gas_price_types::DEFAULT_ETH_TO_FRI_RATE;
2728
use apollo_l1_provider::l1_scraper::L1ScraperConfig;
2829
use apollo_l1_provider::L1ProviderConfig;
@@ -195,6 +196,11 @@ pub fn create_node_config(
195196
};
196197
let override_gas_price_threshold_check = allow_bootstrap_txs;
197198
let mempool_config = create_mempool_config(override_gas_price_threshold_check);
199+
let l1_gas_price_provider_config = L1GasPriceProviderConfig {
200+
// Use newly minted blocks on Anvil to be used for gas price calculations.
201+
lag_margin_seconds: 0,
202+
..Default::default()
203+
};
198204
let http_server_config =
199205
create_http_server_config(available_ports.get_next_local_host_socket());
200206
let class_manager_config =
@@ -246,6 +252,7 @@ pub fn create_node_config(
246252
components: component_config,
247253
l1_scraper_config,
248254
l1_provider_config,
255+
l1_gas_price_provider_config,
249256
..Default::default()
250257
},
251258
config_pointers_map,

0 commit comments

Comments
 (0)