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 Akula to execution client list #7151

Merged
merged 2 commits into from
Jul 27, 2022
Merged

Add Akula to execution client list #7151

merged 2 commits into from
Jul 27, 2022

Conversation

vorot93
Copy link
Contributor

@vorot93 vorot93 commented Jul 21, 2022

No description provided.

@vorot93 vorot93 changed the title Client diversity: add Akula to execution client list Add Akula to execution client list Jul 21, 2022
@gatsby-cloud
Copy link

gatsby-cloud bot commented Jul 21, 2022

Gatsby Cloud Build Report

ethereum-org-website-dev

🎉 Your build was successful! See the Deploy preview here.

Build Details

View the build logs here.

🕐 Build time: 10m

@taxmeifyoucan
Copy link
Member

I would add Akula to Client diversity page with a note that it is experimental or in development. But I would avoid adding to Nodes and clients because this page is being significantly reworked for The Merge #7133 and aims to list only clients ready for it.
This list should only include battle tested, production ready clients usable for an average user, as it states above it "All of them are actively maintained to stay updated with network upgrades, follow current specifications and pass client tests".

@vorot93
Copy link
Contributor Author

vorot93 commented Jul 21, 2022

"Battle-tested, production-ready" is a very vague criteria. On the other hand, Akula syncs on mainnet and all EF-endorsed testnets, including post-merge ones, and passes the consensus test suite.

@wilsoncusack
Copy link

Agree would be great to get more specific about how a client can be added to the nodes and clients page!

@taxmeifyoucan
Copy link
Member

Thanks for clarifying @vorot93! I am running Akula and I can confirm it syncs mainnet and works smoothly, however I wasn't aware Akula is tested, ready for The Merge and considered production ready. With this in mind, I would merge the PR.

Copy link
Contributor

@jmcook1186 jmcook1186 left a comment

Choose a reason for hiding this comment

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

thanks for the PR!

@wackerow wackerow merged commit aba083a into ethereum:dev Jul 27, 2022
@gitpoap-bot
Copy link

gitpoap-bot bot commented Jul 27, 2022

Woohoo, your important contribution to this open-source project has earned you a GitPOAP!

GitPOAP: 2022 Ethereum.org Contributor:

Head on over to GitPOAP.io and connect your GitHub account to mint!

@wackerow
Copy link
Member

@all-contributors please add @vorot93 for content

Thanks for raising this! Be sure to join the discord if you are interested in contributing further to the project or have any questions for the team. You can collect a second POAP (aside from the GitPOAP above) as well =)

@allcontributors
Copy link
Contributor

@wackerow

I've put up a pull request to add @vorot93! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content 🖋️ This involves copy additions or edits
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants