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

Fix: Bump eigensdk-go version #1470

Merged
merged 8 commits into from
Nov 26, 2024
Merged

Conversation

PatStiles
Copy link
Contributor

@PatStiles PatStiles commented Nov 22, 2024

Bump eigensdk-go version

Description

Closes #1468 . Bumps the eigensdk-go version from v1.1.12 -> v1.1.13 .

Note, v1.1.12 contained a library for parsing .yaml and .json files. This library was used in Aligned for starting the variouscomponents of the network. This library was removed in v1.1.13. To work around this breaking change the library was re-implemented within the Aligned Layer repo within core/utils/config_utils.go, the library was ported from this branch of the eigen-sdk repository.

To Test:

  • Run a local devnet and verify proofs land:
  1. make anvil_start_with_block_time
  2. make aggregator_start
  3. make operator_full_registration CONFIG_FILE=config-files/config-operator-1.yaml
  4. make operator_start CONFIG_FILE=config-files/config-operator-1.yaml
  5. make batcher_start_local
  6. make telemetry_full_start

Once you have the system running, send proofs

  1. make batcher_send_burst_groth16
  2. Go to jaeger http://localhost:16686/search
  3. You should see the traces correctly created and task verified
image

Type of change

Please delete options that are not relevant.

  • New feature

Checklist

  • Linked to Github Issue

@PatStiles PatStiles changed the title Feat: Bump eigensdk-go version Fix: Bump eigensdk-go version Nov 22, 2024
Copy link
Collaborator

@MarcosNicolau MarcosNicolau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything works fine, tested on macos.

core/utils/config_utils.go Outdated Show resolved Hide resolved
core/utils/config_utils.go Outdated Show resolved Hide resolved
core/utils/config_utils.go Outdated Show resolved Hide resolved
Copy link
Collaborator

@MarcosNicolau MarcosNicolau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Re-tested with new changes.

@MauroToscano MauroToscano added this pull request to the merge queue Nov 26, 2024
Merged via the queue into staging with commit 8644746 Nov 26, 2024
3 checks passed
@MauroToscano MauroToscano deleted the feat/bump-eigen-sdk-version branch November 26, 2024 20:59
PatStiles added a commit that referenced this pull request Nov 27, 2024
Co-authored-by: MauroFab <[email protected]>
Co-authored-by: Mauro Toscano <[email protected]>
Co-authored-by: Mario Rugiero <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants