-
Notifications
You must be signed in to change notification settings - Fork 9
feat(blueprint-runner/eigenlayer): update eigen sdk #1192
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
…hannel to not block main process
…failed to parse quorum_apks_g1
tracing = { workspace = true } | ||
blueprint-runner = { workspace = true, features = ["networking", "tangle", "std"] } | ||
blueprint-clients = { workspace = true, features = ["std", "tangle"] } | ||
blueprint-runner = { workspace = true, features = ["networking", "tangle", "eigenlayer", "std"] } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be behind a feature flag, tangle + eigenlayer doesn't go together.
|
||
#[inline] | ||
#[must_use] | ||
pub fn eigen_blueprint_binary_path(&self) -> Option<&Path> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why would we need this over the existing method blueprint_config_path
?
@@ -0,0 +1,130 @@ | |||
use crate::config::BlueprintManagerContext; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This entire file seems wrong. There is nothing exactly unique about Eigenlayer events. Eigenlayer is a protocol on EVM that uses Solidity smart contracts or otherwise EVM compatible languages to emit events. The EVM event handling utilities should be sufficient for anything here.
use url::Url; | ||
|
||
// ================= Core Eigenlayer Deployment Addresses ================= | ||
// TODO(daniel): update these addresses to the new addresses of Tangle testnet |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These have nothing to do with Tangle testnet, we care only about anvil local testnet with forked state, the live testnet eigenlayer uses, and mainnet where eigenlayer is deployed. None of these are Tangle.
No description provided.