Skip to content

deployment: add service ports to hybrid environment overlays#14185

Merged
nimrod-starkware merged 2 commits into
mainfrom
05-25-deployment_add_service_ports_to_hybrid_environment_overlays
May 28, 2026
Merged

deployment: add service ports to hybrid environment overlays#14185
nimrod-starkware merged 2 commits into
mainfrom
05-25-deployment_add_service_ports_to_hybrid_environment_overlays

Conversation

@nimrod-starkware
Copy link
Copy Markdown
Contributor

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

@reviewable-StarkWare
Copy link
Copy Markdown

This change is Reviewable

@cursor
Copy link
Copy Markdown

cursor Bot commented May 25, 2026

PR Summary

Low Risk
Deployment-only YAML adding in-cluster service ports; no sequencer logic or security config changes in this diff.

Overview
Adds explicit Kubernetes ClusterIP service definitions to hybrid sequencer overlay configs for mainnet, sepolia-alpha, and sepolia-integration, so in-cluster traffic can reach the right component ports.

For committer, sierra-compiler, and l1 overlays, each file now enables a service block with named ports aligned to existing sequencerConfig component ports (e.g. committer 55013, sierra-compiler 55007; l1 exposes monitoring 55005, gas price 55003, and provider 55004). Gateway overlays additionally publish both the gateway port (55002) and an http-server port (8080).

No application or sequencer tuning values change in this diff—only deployment/service exposure metadata is added across the twelve updated service YAMLs.

Reviewed by Cursor Bugbot for commit 2221ad8. 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 12 files and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on idan-starkware and matanl-starkware).

@nimrod-starkware nimrod-starkware changed the base branch from 05-25-deployment_add_storage_reader_state_sync_network_and_consensus_ports_to_hybrid_environment_overlays to graphite-base/14185 May 28, 2026 05:54
@nimrod-starkware nimrod-starkware force-pushed the 05-25-deployment_add_service_ports_to_hybrid_environment_overlays branch from e20360d to 2221ad8 Compare May 28, 2026 07:07
@nimrod-starkware nimrod-starkware changed the base branch from graphite-base/14185 to 05-25-deployment_add_storage_reader_state_sync_network_and_consensus_ports_to_hybrid_environment_overlays May 28, 2026 07:07
@nimrod-starkware nimrod-starkware changed the base branch from 05-25-deployment_add_storage_reader_state_sync_network_and_consensus_ports_to_hybrid_environment_overlays to main May 28, 2026 07:35
Copy link
Copy Markdown
Contributor Author

@nimrod-starkware nimrod-starkware left a comment

Choose a reason for hiding this comment

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

@nimrod-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).

@nimrod-starkware nimrod-starkware added this pull request to the merge queue May 28, 2026
Merged via the queue into main with commit 4c0e82a May 28, 2026
50 of 63 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