Skip to content

Commit 9fab39e

Browse files
scripts: add Committer to Service options
Add Committer to the Service enum, mapped to its StatefulSet resources (sequencer-committer-config / sequencer-committer-statefulset-0), following the Core pattern since the committer runs as a StatefulSet. It flows automatically into the --service choices, help text, and converter. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent 8b8ce98 commit 9fab39e

2 files changed

Lines changed: 8 additions & 0 deletions

File tree

scripts/prod/common_lib.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -181,6 +181,8 @@ class Service(Enum):
181181
"""Service types mapping to their configmap and pod names."""
182182

183183
Core = ("sequencer-core-config", "sequencer-core-statefulset-0")
184+
# Committer runs as a StatefulSet (like Core), so its pod ends in "-statefulset-0".
185+
Committer = ("sequencer-committer-config", "sequencer-committer-statefulset-0")
184186
Gateway = ("sequencer-gateway-config", "sequencer-gateway-deployment")
185187
L1 = ("sequencer-l1-config", "sequencer-l1-deployment")
186188
Mempool = ("sequencer-mempool-config", "sequencer-mempool-deployment")

scripts/prod/test_enum_cli_tokens.py

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,12 @@ def test_service_str_is_the_accepted_token():
2424
assert Service[str(service)] is service
2525

2626

27+
def test_committer_service_maps_to_statefulset_resources():
28+
# Committer runs as a StatefulSet, so it follows the Core "-statefulset-0" pod pattern.
29+
assert Service.Committer.config_map_name == "sequencer-committer-config"
30+
assert Service.Committer.pod_name == "sequencer-committer-statefulset-0"
31+
32+
2733
def test_invalid_restart_strategy_raises_informative_error():
2834
# Enum value lookup raises ValueError; the converter must translate it to ArgumentTypeError
2935
# with the valid options, rather than letting argparse fall back to a generic message.

0 commit comments

Comments
 (0)