Skip to content

Commit

Permalink
Added correct observability name for builder client
Browse files Browse the repository at this point in the history
  • Loading branch information
SozinM committed Feb 18, 2025
1 parent d96d365 commit ef76286
Showing 1 changed file with 14 additions and 0 deletions.
14 changes: 14 additions & 0 deletions src/el_cl_launcher.star
Original file line number Diff line number Diff line change
Expand Up @@ -300,6 +300,10 @@ def launch(
observability_helper,
interop_params,
)
for metrics_info in [x for x in el_builder_context.el_metrics_info if x != None]:
observability.register_node_metrics_job(
observability_helper, el_builder_context.client_name, "execution-builder", metrics_info
)
rollup_boost_image = (
mev_params.rollup_boost_image
if mev_params.rollup_boost_image != ""
Expand Down Expand Up @@ -370,6 +374,16 @@ def launch(
interop_params,
da_server_context,
)
for metrics_info in [x for x in cl_builder_context.cl_nodes_metrics_info if x != None]:
observability.register_node_metrics_job(
observability_helper,
cl_builder_context.client_name,
"beacon-builder",
metrics_info,
{
"supernode": str(cl_builder_context.supernode),
},
)
all_cl_contexts.append(cl_builder_context)

# We need to make sure that el_context and cl_context are first in the list, as down the line all_el_contexts[0]
Expand Down

0 comments on commit ef76286

Please sign in to comment.