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,ci: add hybrid native override layers and their validation tests #14610
• deployment: add native node-config builder and wire it into the ConfigMap #14609
• 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 : 1 other dependent PR (#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.