deployment: thread config_format from CLI to node container arg

[nimrod-starkware]

Add a --config-format {preset,native} deployment CLI flag (default preset) and
thread its value from the parser through SequencerNodeChart and the
Deployment/StatefulSet constructs to PodBuilder, which now emits
--config_format <value> to the node container instead of a hardcoded "preset".
With the default (preset) the synthesized output is unchanged. The ConfigMap
content path (native config generation) consumes this value in a follow-up.

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

[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 👈 (View in Graphite)
• apollo_deployments,deployment: set storage-reader-server ports as constants #14607
• 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.

[reviewable-StarkWare]

This change is 

[nimrod-starkware]

@nimrod-starkware made 2 comments.
Reviewable status: 0 of 6 files reviewed, 2 unresolved discussions.

---

deployments/sequencer/src/charts/node.py line 34 at r1 (raw file):

        layout: str,
        overlays: list[str],
        config_format: str = "preset",

no default value, remove

Code quote:

config_format: str = "preset",

---

deployments/sequencer/src/constructs/deployment.py line 14 at r1 (raw file):

        labels,
        monitoring_endpoint_port,
        config_format: str = "preset",

no default value, remove

Code quote:

config_format: str = "preset",

[nimrod-starkware]

@nimrod-starkware resolved 2 discussions.
Reviewable status: 0 of 6 files reviewed, all discussions resolved.

[cursor]

PR Summary

Low Risk
Default remains preset so existing synth output is unchanged; only non-default native changes runtime config loading, and ConfigMap generation is not wired yet.

Overview
Adds --config-format (preset | native, default preset) to the sequencer deployment CLI so operators can choose how the node loads config.

The value flows from app.py → SequencerNodeChart → Deployment/StatefulSet constructs → PodBuilder, which now passes --config_format <value> to the container instead of always preset. With the default, synthesized manifests match prior behavior.

ConfigMap content for native is explicitly out of scope here (follow-up PR).

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

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 528af92. Configure here.}