deployment: thread config_format from CLI to node container arg

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>

Author: nimrod-starkware

deployments/sequencer/src/app.py

@@ -49,6 +49,7 @@ def main():
             monitoring_configs["enabled"],
             args.layout,
             args.overlay or [],
+            args.config_format,
         )
         _create_monitoring_chart(app, namespace, args.cluster, monitoring_configs)
 
@@ -199,6 +200,7 @@ def _create_service_charts(
     monitoring_enabled: bool,
     layout: str,
     overlays: list[str],
+    config_format: str,
 ):
     """Create SequencerNodeChart for each service."""
     for service_cfg in deployment_config.services:
@@ -210,6 +212,7 @@ def _create_service_charts(
             service_config=service_cfg,
             layout=layout,
             overlays=overlays,
+            config_format=config_format,
         )
 
 

deployments/sequencer/src/charts/node.py

@@ -31,11 +31,13 @@ def __init__(
         service_config: ServiceConfig,
         layout: str,
         overlays: list[str],
+        config_format: str,
     ):
         super().__init__(scope, name, disable_resource_name_hashes=True, namespace=namespace)
 
         self.monitoring = monitoring
         self.service_config = service_config
+        self.config_format = config_format
 
         # Create labels dictionary from service config + service name
         # Base labels from shared config (metaLabels) - now merged into service_config
@@ -48,7 +50,13 @@ def __init__(
 
         # Create ConfigMap
         self.config_map = ConfigMapConstruct(
-            self, "configmap", service_config, labels, monitoring_endpoint_port, layout, overlays
+            self,
+            "configmap",
+            service_config,
+            labels,
+            monitoring_endpoint_port,
+            layout,
+            overlays,
         )
 
         # Create ServiceAccount if enabled
@@ -95,6 +103,7 @@ def __init__(
                 service_config=self.service_config,
                 labels=labels,
                 monitoring_endpoint_port=monitoring_endpoint_port,
+                config_format=config_format,
             )
         else:
             self.controller = DeploymentConstruct(
@@ -103,6 +112,7 @@ def __init__(
                 service_config=self.service_config,
                 labels=labels,
                 monitoring_endpoint_port=monitoring_endpoint_port,
+                config_format=config_format,
             )
 
         # Create BackendConfig if enabled

deployments/sequencer/src/cli.py

@@ -75,6 +75,16 @@ def argument_parser():
         action=UniqueStoreAction,
         help="Override image for all services. Format: 'repository:tag' or 'repository' (defaults to 'latest' tag).",
     )
+    parser.add_argument(
+        "--config-format",
+        type=str,
+        action=UniqueStoreAction,
+        choices=["preset", "native"],
+        default="preset",
+        help="Node config format to emit and pass to the node container via --config_format. "
+        "'preset' (default): flat dotted-key placeholder fill (legacy). "
+        "'native': nested SequencerNodeConfig assembled via jsonnet build().",
+    )
     parser.add_argument(
         "-v",
         "--verbose",

deployments/sequencer/src/constructs/deployment.py

@@ -11,6 +11,7 @@ def __init__(
         service_config,
         labels,
         monitoring_endpoint_port,
+        config_format: str,
     ):
         super().__init__(
             scope,
@@ -20,13 +21,15 @@ def __init__(
             monitoring_endpoint_port,
         )
 
+        self.config_format = config_format
         self.deployment = self._create_deployment()
 
     def _create_deployment(self) -> k8s.KubeDeployment:
         pod_builder = PodBuilder(
             self.service_config,
             self.labels,
             self.monitoring_endpoint_port,
+            config_format=self.config_format,
         )
 
         selector_labels = {"service": self.labels["service"]}

deployments/sequencer/src/constructs/helpers/pod_builder.py

@@ -16,10 +16,12 @@ def __init__(
         service_config: ServiceConfig,
         labels: dict[str, str],
         monitoring_endpoint_port: int,
+        config_format: str,
     ):
         self.service_config = service_config
         self.labels = labels
         self.monitoring_endpoint_port = monitoring_endpoint_port
+        self.config_format = config_format
 
     def build_pod_spec(self) -> k8s.PodSpec:
         """Build a complete PodSpec with all necessary configurations."""
@@ -82,9 +84,8 @@ def _build_container(self) -> k8s.Container:
 
     def _build_container_args(self) -> list[str]:
         """Build container arguments, always including --config_file with fixed file paths."""
-        # Interpret the --config_file arguments with the legacy flat-preset loader for now. The
-        # node also supports "native" (nested serde) loading; this is the explicit switch point.
-        args = ["--config_format", "preset"]
+        # First argument is the config format.
+        args = ["--config_format", self.config_format]
 
         # Add --config_file /config/sequencer/presets/config (ConfigMap)
         # Note: node version uses directory mount, so path is just the directory + "config"

deployments/sequencer/src/constructs/statefulset.py

@@ -11,6 +11,7 @@ def __init__(
         service_config,
         labels,
         monitoring_endpoint_port,
+        config_format,
     ):
         super().__init__(
             scope,
@@ -20,6 +21,7 @@ def __init__(
             monitoring_endpoint_port,
         )
 
+        self.config_format = config_format
         self.statefulset = self._create_statefulset()
 
     def _build_update_strategy(self) -> k8s.StatefulSetUpdateStrategy:
@@ -67,6 +69,7 @@ def _create_statefulset(self) -> k8s.KubeStatefulSet:
             self.service_config,
             statefulset_labels,
             self.monitoring_endpoint_port,
+            config_format=self.config_format,
         )
 
         selector_labels = {"service": statefulset_labels["service"]}