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

Add Lodestar #40

Closed
wants to merge 22 commits into from
Closed

Add Lodestar #40

wants to merge 22 commits into from

Conversation

dgcoffman
Copy link
Collaborator

@dgcoffman dgcoffman commented Oct 18, 2022

  1. Integrates Lodestar from EIP-4844 dgcoffman/lodestar#1
  2. Modifies the existing tests to pass against Prysm without using its proprietary gRPC API, so the same tests can be run against Lodestar.

TODO

  • Add Lodestar directory, git submodule, chain config, Docker setup
  • Existing go run ./tests/... tests all pass
  • go run ./tests/pre-4844 lodestar passes

@dgcoffman dgcoffman mentioned this pull request Oct 19, 2022
25 tasks
@roberto-bayardo
Copy link
Collaborator

I know this is still a WIP but you might submit this PR against the recently-created devnet-v3 branch instead of master?

@dgcoffman dgcoffman changed the base branch from master to devnet-v3 October 31, 2022 16:57
@dgcoffman dgcoffman changed the base branch from devnet-v3 to master October 31, 2022 16:58
@dgcoffman dgcoffman requested a review from Inphi November 10, 2022 21:49
@dgcoffman dgcoffman marked this pull request as ready for review November 10, 2022 21:49
@Inphi
Copy link
Owner

Inphi commented Nov 10, 2022

Seconded @roberto-bayardo . Let's have this target the devnet-v3 branch.

@roberto-bayardo
Copy link
Collaborator

keep in mind that we haven't updated prysm/geth for the latest (3038) protocol changes so it might not play nice just yet with the other clients. I'll get that done soon though.

@dgcoffman
Copy link
Collaborator Author

#52

@dgcoffman dgcoffman closed this Nov 10, 2022
@dgcoffman dgcoffman deleted the dgc/run-lodestar-nodes branch November 10, 2022 22:40
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.

3 participants