apollo_deployments,deployment: set storage-reader-server ports as constants

[nimrod-starkware]

The three storage-reader-server ports (batcher 55011, class_manager 55210,
state_sync 55014) are fixed infra constants, not per-deployment overrides. Bake
them into the app_configs and applicative_config jsonnet instead of templatizing
them as $$$...$$$ placeholders, and restrict the generic port/url
auto-templatizing in replace_pred to components.* keys only. Add
state_sync_config...network_config.port to KEYS_TO_BE_REPLACED to preserve its
existing overridability. Drop the now-redundant storage-reader port overrides
from the common and testing overlay YAMLs.

Co-Authored-By: Claude Opus 4.8 (1M context) noreply@anthropic.com

[reviewable-StarkWare]

This change is 

[nimrod-starkware]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• deployment: add native node-config builder and wire it into the ConfigMap #14609 : 2 dependent PRs (#14655 , #14674 )
• deployment: thread config_format from CLI to node container arg #14608
• apollo_deployments,deployment: set storage-reader-server ports as constants #14607 👈 (View in Graphite)
• apollo_node: add new CLI arg to parse config #14606
• apollo_deployments: check applicative config matches deployed app_configs #14489 : 2 other dependent PRs (#14480 , #14505 )
• apollo_deployments: assemble and deserialize node config from jsonnet #14487
• apollo_deployments: assemble full node config via build.libsonnet #14486
• apollo_deployments: distributed layout jsonnet test #14485
• apollo_deployments: consolidated layout jsonnet test #14484
• apollo_deployments: test hybrid layout for infra config #14483
• apollo_deployments: introduce hybrid layout for infra config #14482
• apollo_deployments: introduce jsonnet building blocks for infra config #14481
• apollo_deployments: jsonnet applicative config #14479
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[nimrod-starkware]

@nimrod-starkware made 1 comment.
Reviewable status: 0 of 10 files reviewed, 1 unresolved discussion.

---

crates/apollo_deployments/src/service.rs line 369 at r1 (raw file):

    // auto-templatized — an overridable one must be listed explicitly in `KEYS_TO_BE_REPLACED`;
    // the rest (e.g. the fixed storage-reader-server ports) keep their literal app_config
    // value.

shrink comment

Code quote:

    // Only `components.*` infra fields are auto-templatized: the layout assigns each component's
    // remote URL and port per deployment. Ports/URLs OUTSIDE `components.*` are NOT
    // auto-templatized — an overridable one must be listed explicitly in `KEYS_TO_BE_REPLACED`;
    // the rest (e.g. the fixed storage-reader-server ports) keep their literal app_config
    // value.

[nimrod-starkware]

@nimrod-starkware resolved 1 discussion.
Reviewable status: 0 of 10 files reviewed, all discussions resolved.

[nimrod-starkware]

@nimrod-starkware made 1 comment.
Reviewable status: 0 of 10 files reviewed, 1 unresolved discussion.

---

crates/apollo_deployments/src/service.rs line 103 at r2 (raw file):

    "state_sync_config.static_config.central_sync_client_config.sync_config.store_sierras_and_casms_block_threshold",
    "state_sync_config.static_config.network_config.#is_none",
    "state_sync_config.static_config.network_config.port",

is this needed?

Code quote:

 "state_sync_config.static_config.network_config.port"

[cursor]

PR Summary

Medium Risk
Fixed listen ports must match K8s service exposure; misalignment would break storage-reader connectivity, though values align with existing hybrid service port definitions.

Overview
Storage-reader-server ports are no longer deployment overrides or $$$_..._$$$ replacer placeholders. Batcher (55011), class manager (55210), and state sync (55014) are set directly in applicative jsonnet and app_configs (replacing the old shared 8091 default).

Replacer generation in service.rs now only auto-templatizes .port / .url keys under components.*; applicative ports like storage readers stay literal. state_sync_config.static_config.network_config.port is added to KEYS_TO_BE_REPLACED so P2P network port overrides stay explicit.

Hybrid core overlay YAML drops the redundant storage-reader port entries from sequencerConfig (Kubernetes service.ports for those listeners is unchanged).

^{Reviewed by Cursor Bugbot for commit 6ca2fa5. Bugbot is set up for automated code reviews on this repo. Configure here.}