Skip to content

deployment: add component ports and urls to hybrid environment overlays#14181

Merged
nimrod-starkware merged 1 commit into
mainfrom
05-25-deployment_add_component_ports_and_urls_to_hybrid_environment_overlays
May 27, 2026
Merged

deployment: add component ports and urls to hybrid environment overlays#14181
nimrod-starkware merged 1 commit into
mainfrom
05-25-deployment_add_component_ports_and_urls_to_hybrid_environment_overlays

Conversation

@nimrod-starkware
Copy link
Copy Markdown
Contributor

@nimrod-starkware nimrod-starkware commented May 25, 2026

Add components.* port and URL entries to each production environment's service overlays (core, l1, gateway, mempool, committer, sierra-compiler) for mainnet, sepolia-alpha, and sepolia-integration.

Values are identical across all three environments.

Corresponding devops PR (removes the same keys from the devops overlays): starkware-industries/sequencer-devops#703

@reviewable-StarkWare
Copy link
Copy Markdown

This change is Reviewable

@nimrod-starkware nimrod-starkware force-pushed the 05-25-deployment_add_component_ports_and_urls_to_hybrid_environment_overlays branch from 924d0ed to 1c33c4c Compare May 25, 2026 07:44
@nimrod-starkware nimrod-starkware force-pushed the 05-25-deployment_add_recorder_url_to_hybrid_environment_overlays branch from 0e34ea9 to 1bc33cd Compare May 25, 2026 07:44
@nimrod-starkware nimrod-starkware marked this pull request as ready for review May 25, 2026 07:53
@nimrod-starkware nimrod-starkware self-assigned this May 25, 2026
@cursor
Copy link
Copy Markdown

cursor Bot commented May 25, 2026

PR Summary

Medium Risk
Wrong host/port wiring would break inter-service RPC at deploy time across production networks; changes are config-only but affect live hybrid topology.

Overview
Adds components.*.port and components.*.url entries under sequencerConfig in hybrid deployment service overlays for mainnet, sepolia-alpha, and sepolia-integration.

Each service file (core, l1, gateway, mempool, committer, sierra-compiler) now lists only the component endpoints that pod needs (e.g. core gets the full cross-service map; committer gets batcher + committer). Hostnames follow the sequencer-*-service pattern and ports use the 55000–55013 range; the same values are repeated across all three networks.

This moves wiring that was in devops overlays into this repo (paired with devops PR that removes those keys).

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

Copy link
Copy Markdown
Collaborator

@dorimedini-starkware dorimedini-starkware left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dorimedini-starkware reviewed 18 files and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on idan-starkware and matanl-starkware).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@nimrod-starkware nimrod-starkware force-pushed the 05-25-deployment_add_component_ports_and_urls_to_hybrid_environment_overlays branch from 1c33c4c to ea09641 Compare May 27, 2026 06:27
@nimrod-starkware nimrod-starkware changed the base branch from graphite-base/14181 to main May 27, 2026 06:27
Copy link
Copy Markdown
Collaborator

@dorimedini-starkware dorimedini-starkware left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dorimedini-starkware reviewed 3 files and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on idan-starkware and matanl-starkware).

Merged via the queue into main with commit 225e40f May 27, 2026
56 of 76 checks passed
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.

3 participants