Skip to content

apollo_consensus_orchestrator_config: add fetch_accessed_keys_from_centralized config #14820

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.

Sign up for GitHub

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

Draft
yoavGrs wants to merge 1 commit into
base: cende-delta-1-fetch-helper
Choose a base branch
from
+16 −0
Draft

apollo_consensus_orchestrator_config: add fetch_accessed_keys_from_centralized config #14820

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 11 additions & 0 deletions crates/apollo_consensus_orchestrator_config/src/config.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -177,6 +177,9 @@ pub struct ContextStaticConfig {
#[serde(deserialize_with = "deserialize_milliseconds_to_duration")]
pub retrospective_block_hash_retry_interval_millis: Duration,
pub behavior_mode: BehaviorMode,
/// When adding a synced block, fetch its accessed keys from the recorder so the node can build
/// the state commitment infos locally. Only has an effect in `os_input` builds.
pub fetch_accessed_keys_from_centralized: bool,
}

impl SerializeConfig for ContextStaticConfig {
Expand Down Expand Up @@ -241,6 +244,13 @@ impl SerializeConfig for ContextStaticConfig {
"Behavior mode: 'starknet' for production, 'echonet' for test/replay mode.",
ParamPrivacyInput::Public,
)]);
dump.extend([ser_param(
"fetch_accessed_keys_from_centralized",
&self.fetch_accessed_keys_from_centralized,
"Fetch accessed keys from the centralized recorder for synced blocks, enabling local \
state commitment infos construction. Only has an effect in `os_input` builds.",
ParamPrivacyInput::Public,
)]);
dump
}
}
Expand All @@ -257,6 +267,7 @@ impl Default for ContextStaticConfig {
build_proposal_time_ratio_for_retrospective_block_hash: 0.7,
retrospective_block_hash_retry_interval_millis: Duration::from_millis(500),
behavior_mode: BehaviorMode::default(),
fetch_accessed_keys_from_centralized: false,
}
}
}
Expand Down
5 changes: 5 additions & 0 deletions crates/apollo_node/resources/config_schema.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2804,6 +2804,11 @@
"pointer_target": "chain_id",
"privacy": "Public"
},
"consensus_manager_config.context_config.static_config.fetch_accessed_keys_from_centralized": {
"description": "Fetch accessed keys from the centralized recorder for synced blocks, enabling local state commitment infos construction. Only has an effect in `os_input` builds.",
"privacy": "Public",
"value": false
},
"consensus_manager_config.context_config.static_config.l1_da_mode": {
"description": "The data availability mode, true: Blob, false: Calldata.",
"privacy": "Public",
Expand Down
Loading