Skip to content

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

Open
nimrod-starkware wants to merge 1 commit into
graphite-base/14181from
05-25-deployment_add_component_ports_and_urls_to_hybrid_environment_overlays
Open

deployment: add component ports and urls to hybrid environment overlays#14181
nimrod-starkware wants to merge 1 commit into
graphite-base/14181from
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

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 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 cross-pod RPC at rollout; changes are repetitive config only with no application logic, but must deploy in sync with the devops overlay removal.

Overview
Adds components.*.port and components.*.url under sequencerConfig in hybrid deployment overlays for mainnet, sepolia-alpha, and sepolia-integration, wiring each split service to the correct Kubernetes service hostnames (e.g. sequencer-core-service, sequencer-mempool-service) on ports 55000–55013.

Each service overlay only gets the component endpoints that process needs (e.g. core gets the full cross-service map; committer only batcher + committer; sierra-compiler only sierra_compiler). Host/port values are the same across all three environments; only existing env-specific settings (L1 addresses, cache sizes, etc.) stay different.

This moves hybrid inter-service discovery into the sequencer repo overlays (paired with removal of the same keys from sequencer-devops).

Reviewed by Cursor Bugbot for commit 1c33c4c. 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).

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