Skip to content

Commit 312f0f3

Browse files
chetanybzeam-bot
authored andcommitted
fix: update zeam CLI flags for API/metrics server separation
1 parent 9b910e1 commit 312f0f3

2 files changed

Lines changed: 14 additions & 3 deletions

File tree

ansible/roles/zeam/tasks/main.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,6 +89,9 @@
8989
- name: Start Zeam container
9090
# TODO: Remove --platform linux/amd64 when blockblaz/zeam:latest multi-platform image is available on Docker Hub
9191
# Multi-platform support is being added in zeam CI (see .github/workflows/ci.yml docker-build-multiarch job)
92+
# Zeam runs two HTTP servers:
93+
# - API server (default 9667): health, SSE events, forkchoice graph, checkpoint state
94+
# - Metrics server (--metrics-port): Prometheus /metrics endpoint
9295
command: >-
9396
docker run -d
9497
--pull=always
@@ -108,7 +111,7 @@
108111
--node-id {{ node_name }}
109112
--node-key /config/{{ node_name }}.key
110113
--metrics_enable
111-
--api-port {{ zeam_metrics_port }}
114+
--metrics-port {{ zeam_metrics_port }}
112115
{{ '--is-aggregator' if (zeam_is_aggregator | default('false')) == 'true' else '' }}
113116
{{ ('--checkpoint-sync-url ' + checkpoint_sync_url) if (checkpoint_sync_url is defined and checkpoint_sync_url | length > 0) else '' }}
114117
register: zeam_container_result

client-cmds/zeam-cmd.sh

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,14 @@
33
#-----------------------zeam setup----------------------
44
# setup where lean-quickstart is a submodule folder in zeam repo
55
# update the path to your binary here if you want to use binary
6+
#
7+
# Zeam runs two HTTP servers:
8+
# - API server (default port 9667): health, SSE events, forkchoice graph, checkpoint state
9+
# - Metrics server (--metrics-port): Prometheus /metrics endpoint
10+
#
11+
# The metricsPort from validator-config.yaml is used for the metrics server.
12+
# API server uses the default port 9667 (override with --api-port if needed).
13+
#
614
# Metrics enabled by default
715
metrics_flag="--metrics_enable"
816

@@ -30,7 +38,7 @@ node_binary="$scriptDir/../zig-out/bin/zeam node \
3038
--data-dir $dataDir/$item \
3139
--node-id $item --node-key $configDir/$item.key \
3240
$metrics_flag \
33-
--api-port $metricsPort \
41+
--metrics-port $metricsPort \
3442
$attestation_committee_flag \
3543
$aggregator_flag \
3644
$checkpoint_sync_flag"
@@ -41,7 +49,7 @@ node_docker="--security-opt seccomp=unconfined blockblaz/zeam:devnet3 node \
4149
--data-dir /data \
4250
--node-id $item --node-key /config/$item.key \
4351
$metrics_flag \
44-
--api-port $metricsPort \
52+
--metrics-port $metricsPort \
4553
$attestation_committee_flag \
4654
$aggregator_flag \
4755
$checkpoint_sync_flag"

0 commit comments

Comments
 (0)