Skip to content
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

HyperlaneServiceManager #3590

Closed
Tracked by #3588
aroralanuk opened this issue Apr 12, 2024 · 0 comments · Fixed by #3651
Closed
Tracked by #3588

HyperlaneServiceManager #3590

aroralanuk opened this issue Apr 12, 2024 · 0 comments · Fixed by #3651
Assignees

Comments

@aroralanuk
Copy link
Contributor

aroralanuk commented Apr 12, 2024

Implement HyperlaneServiceManager as IServiceManager, the main AVS contract surface area as per https://github.com/hyperlane-xyz/hips/blob/kunal/el/HIP-8.md

@aroralanuk aroralanuk self-assigned this Apr 13, 2024
@aroralanuk aroralanuk moved this to Sprint in Hyperlane Tasks Apr 13, 2024
@avious00 avious00 moved this from Sprint to Next Sprint in Hyperlane Tasks Apr 15, 2024
@nambrot nambrot moved this from Next Sprint to Sprint in Hyperlane Tasks Apr 16, 2024
@avious00 avious00 moved this from Sprint to In Progress in Hyperlane Tasks Apr 23, 2024
@avious00 avious00 moved this from In Progress to In Review in Hyperlane Tasks May 8, 2024
aroralanuk added a commit that referenced this issue May 9, 2024
### Description

- Implements `IServiceManager` for the `ECDSAServiceManagerBase` which
has a single quorum
- `HyperlaneServiceManager` (HSM), the main AVS contract for
registering/deregistering operators and enrolling/unenrolling into
challengers
- the HSM has a mutable slasher so that once the EL slasher is live, we
can set it for your AVS too to enable freezing operators.
- EnumerableMapEnrollment for a enumerable map of enrollment statuses
for each operator and challenger

Update: I switched the ECDSAServiceManager i had written earlier to
steven's PR here:
Layr-Labs/eigenlayer-middleware#234. The
differences are:
 - events for (de)registering

Notes

- `hardhat yarn` tries looking for all imports (including the ones from
submodules) as npm libraries , eg. @eigenlayer/core.

### Drive-by changes

- none

### Related issues

- fixes #3590,

### Backward compatibility

<!--
Are these changes backward compatible? Are there any infrastructure
implications, e.g. changes that would prohibit deploying older commits
using this infra tooling?

Yes/No
-->

### Testing

<!--
What kind of testing have these changes undergone?

None/Manual/Unit Tests
-->
@github-project-automation github-project-automation bot moved this from In Review to Done in Hyperlane Tasks May 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant