feat(telemetry): Add url data to operator updates on telemetry service#1136
feat(telemetry): Add url data to operator updates on telemetry service#1136
Conversation
feat: CRUD for operators version
docs: update README
… creation (#1080) Co-authored-by: Julian Ventura <julian.ventura@lambdaclass.com>
Co-authored-by: Julian Ventura <julian.ventura@lambdaclass.com>
…-in-elixir' into 1075-feattelemetry-implement-telemetry-for-aggregator # Conflicts: # telemetry_api/config/dev.exs # telemetry_api/mix.exs # telemetry_api/mix.lock
…erator_tracker-in-elixir
…-in-elixir' into 1075-feattelemetry-implement-telemetry-for-aggregator
fix: store address and operator_id in lowercase
…-in-elixir' into 1075-feattelemetry-implement-telemetry-for-aggregator
Changes to gas cost
🧾 Summary (10% most significant diffs)
Full diff report 👇
|
MarcosNicolau
left a comment
There was a problem hiding this comment.
Tested on my machine, and it worked fine. One thing, in the diff I see you changed the eigenlayer-middleware, is that on purpose?
No, I haven't touched anything related to that. There was a merge from staging branch, so maybe it was an auto merge error? |
My bad. The submodule commit was in fact different from the one in staging branch. Good catch! |
|
|
uri-99
left a comment
There was a problem hiding this comment.
Meant to request changes because of my previous comments. Everything else works
Description
This PR adds new data to be updated for each operator on the telemetry service, which includes:
eth_rpc_urleth_rpc_url_fallbacketh_ws_urleth_ws_url_fallbackversion(this one was already present)The implementation is also updated from the operator's side.
Testing
In order to test, the following steps were taken:
make anvil_start_with_block_timemake aggregator_startmake build_operatormake operator_full_registration CONFIG_FILE=config-files/config-operator-1.yamlmake telemetry_clean_dbmake telemetry_run_dbmake telemetry_starthttp://localhost:4001/api/operatorsand you should see the operator with its data fields. Some of them, like the version and RPC related should be set as null, since the operator is not up yet../operator/build/aligned-operator start --config ./config-files/config-operator-1.yamlhttp://localhost:4001/api/operatorsand you should see the operator, now with all fields with a certain value, including version and RPC related.It is also highly advisable to test previous functionality as it's described on this already merged PR