Skip to content

deployment: extract sierra-compiler common config#14243

Open
nimrod-starkware wants to merge 1 commit into
05-28-deployment_support_include_directive_in_common_yaml_loaderfrom
05-28-deployment_extract_sierra-compiler_common_config
Open

deployment: extract sierra-compiler common config#14243
nimrod-starkware wants to merge 1 commit into
05-28-deployment_support_include_directive_in_common_yaml_loaderfrom
05-28-deployment_extract_sierra-compiler_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:24
@cursor
Copy link
Copy Markdown

cursor Bot commented May 28, 2026

PR Summary

Low Risk
YAML-only refactor with per-environment overrides preserved; verify overlay merge still applies audited_libfuncs_only correctly at deploy time.

Overview
Extracts duplicated sierra-compiler hybrid sequencer service settings into configs/overlays/hybrid/common/services/sierra-compiler.yaml, covering the shared configList, compiler port/URL, max_bytecode_size, and ClusterIP service on port 55007.

mainnet, sepolia-alpha, and sepolia-integration overlays now include that common file and only set sierra_compiler_config.audited_libfuncs_only (true on mainnet and sepolia-alpha, false on sepolia-integration).

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