Skip to content

deployment: extract committer common config#14244

Open
nimrod-starkware wants to merge 1 commit into
05-28-deployment_extract_sierra-compiler_common_configfrom
05-28-deployment_extract_committer_common_config
Open

deployment: extract committer common config#14244
nimrod-starkware wants to merge 1 commit into
05-28-deployment_extract_sierra-compiler_common_configfrom
05-28-deployment_extract_committer_common_config

Conversation

@nimrod-starkware
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor Author

nimrod-starkware commented May 28, 2026

@reviewable-StarkWare
Copy link
Copy Markdown

This change is Reviewable

@nimrod-starkware nimrod-starkware self-assigned this May 28, 2026
@nimrod-starkware nimrod-starkware marked this pull request as ready for review May 28, 2026 13:25
@cursor
Copy link
Copy Markdown

cursor Bot commented May 28, 2026

PR Summary

Low Risk
Deployment config deduplication with per-env cache sizes unchanged in intent; no application or auth logic changes.

Overview
Introduces a shared hybrid committer service overlay at configs/overlays/hybrid/common/services/committer.yaml and wires mainnet, sepolia-alpha, and sepolia-integration to it via include, instead of duplicating the same service definition in each environment.

The common file holds shared sequencer settings (config list, verify_state_diff_hash, batcher/committer URLs and ports) and the ClusterIP service on port 55013. Each environment overlay now only overrides committer_config.storage_config.cache_size (50M on mainnet, 10M on the Sepolia overlays).

Reviewed by Cursor Bugbot for commit 2893ae6. 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