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.

Sign up for GitHub

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

Jump to bottom

Upgrading sdk version to 0.1.12 #86

Merged
merged 32 commits into from
Nov 15, 2024
Merged

Upgrading sdk version to 0.1.12 #86

merged 32 commits into from
Nov 15, 2024

Conversation

pablodeymo
Copy link
Contributor

@pablodeymo pablodeymo commented Oct 8, 2024 •
edited by TomasArrachea
Loading

Upgraded the go-sdk version to 0.1.12 and made the necessary changes in the AVS logic:

  • Operators now send the task response instead of the digest
  • The aggregator receives a custom hash function as parameter (it must be the same as the one using on the operator to sign the task response)
  • Updated interfaces to use InstrumentedClient

@pablodeymo pablodeymo changed the title [WIP] Upgrading sdk version i go.mod and go.sum [WIP] Upgrading sdk version Oct 8, 2024
@pablodeymo
BuildAvsSubscriber fixed
c2b5236
@pablodeymo
adding missing parameter to Send
18fb982
@pablodeymo
changing to logger := testutils.GetTestLogger()
4f47565
@pablodeymo
challenger fixed
e65fbde
@pablodeymo
hashFunction in aggregator.go
e1d4e35
@pablodeymo
MockBlsAggregationService in aggregator_test.go
ab045cf
@pablodeymo
fixing config.go
bb10b73
@pablodeymo
fixing avs_reader avs_writer y bindings
588a6a5
@pablodeymo
fixing avs writer and reader
8a16a93
@pablodeymo
AvsWriterer and AvsReaderer fixed
74e13e9
@pablodeymo
operator.go fixed
253ea2b
@pablodeymo
registration.go fixed
5fdb7d7
@pablodeymo
aggregator.go fixed
da50d14
@pablodeymo
RegisterOperatorInQuorumWithAVSRegistryCoordinator in main.go fixed
4880aaf
@pablodeymo
main.go fixed
db0e8c7
@pablodeymo
integration_test.go fixed
1210835
@pablodeymo
challenger fixed
fef94ea
@pablodeymo
Adding interface EthClientInterface
4e03208
@pablodeymo
Merge branch 'master' into upgrade_to_eigen_sdk_0.1.12
d862586
@pablodeymo
fix errors with pointers
e607aba
@pablodeymo
aggregator hash function fixed to match the one used by the operator
cb9225b
@pablodeymo pablodeymo marked this pull request as ready for review October 15, 2024 14:12
@pablodeymo pablodeymo changed the title [WIP] Upgrading sdk version Upgrading sdk version to 0.1.12 Oct 15, 2024
@stevennevins stevennevins mentioned this pull request Oct 17, 2024
Closed
@TomasArrachea TomasArrachea marked this pull request as draft October 17, 2024 21:14
@TomasArrachea TomasArrachea marked this pull request as ready for review October 18, 2024 15:52
@pablodeymo pablodeymo merged commit a7cc616 into master Nov 15, 2024
5 checks passed
@pablodeymo pablodeymo deleted the upgrade_to_eigen_sdk_0.1.12 branch November 15, 2024 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stevennevins stevennevins stevennevins approved these changes

@afkbyte afkbyte Awaiting requested review from afkbyte

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@pablodeymo @stevennevins @TomasArrachea