Skip to content

deployment: add mainnet native config override layer#14628

Open
nimrod-starkware wants to merge 1 commit into
nimrod/jsonnet/native-override-alphafrom
nimrod/jsonnet/native-override-mainnet
Open

deployment: add mainnet native config override layer#14628
nimrod-starkware wants to merge 1 commit into
nimrod/jsonnet/native-override-alphafrom
nimrod/jsonnet/native-override-mainnet

Conversation

@nimrod-starkware

Copy link
Copy Markdown
Contributor

Translate the mainnet config.sequencerConfig overrides (the public common.yaml +
services/*.yaml flat dotted keys) into the nested sequencer_config.jsonnet
override layer the native config builder consumes, mirroring the existing
sepolia-integration and sepolia-alpha layers. Add a regression test asserting
the jsonnet layer stays in lockstep with the combined YAML overrides (folded:
#is_none applied, components dropped).

Co-Authored-By: Claude Opus 4.8 (1M context) noreply@anthropic.com

nimrod-starkware commented Jun 25, 2026

Copy link
Copy Markdown
Contributor Author

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@reviewable-StarkWare

Copy link
Copy Markdown

This change is Reviewable

@nimrod-starkware nimrod-starkware force-pushed the nimrod/jsonnet/native-override-mainnet branch from 652ee03 to 2e11c32 Compare June 28, 2026 08:19
@nimrod-starkware nimrod-starkware force-pushed the nimrod/jsonnet/native-override-alpha branch from da1af43 to 52a1181 Compare June 28, 2026 08:19
@nimrod-starkware nimrod-starkware force-pushed the nimrod/jsonnet/native-override-alpha branch from 52a1181 to cc2b14f Compare June 28, 2026 10:14
@nimrod-starkware nimrod-starkware force-pushed the nimrod/jsonnet/native-override-mainnet branch from 2e11c32 to aa981cd Compare June 28, 2026 10:14
@nimrod-starkware nimrod-starkware force-pushed the nimrod/jsonnet/native-override-mainnet branch 2 times, most recently from cfcf68f to ab93940 Compare June 28, 2026 14:04
@cursor

cursor Bot commented Jun 29, 2026

Copy link
Copy Markdown

PR Summary

Medium Risk
Mainnet production sequencer parameters are involved; mis-translation could change consensus, sync, or batching behavior, though the parity test limits drift from the existing YAML source of truth.

Overview
Adds a mainnet hybrid sequencer overlay in native Jsonnet (chain_params.jsonnet + replacers.jsonnet), matching the pattern used for sepolia-integration and sepolia-alpha.

chain_params.jsonnet carries mainnet-specific chain identity and infrastructure: SN_MAIN, feeder/recorder URLs, native class whitelist, L1 base-layer block thresholds and contract address, default staking committee, proof-archive GCS bucket, and the partial-block-hash migration anchor block.

replacers.jsonnet holds only non-default tunables from the YAML layer (bouncer state-diff cap, execute workers, committer cache size, height-indexed min L2 gas price, state-sync Sierra/CASM storage threshold, and state-sync network port).

A new test_mainnet_native_matches_yaml parity test ensures the bucketed Jsonnet stays aligned with folded config.sequencerConfig from common.yaml and services/*.yaml.

Reviewed by Cursor Bugbot for commit 84a8fbc. Bugbot is set up for automated code reviews on this repo. Configure here.

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.

2 participants