Skip to content

Conversation

kevaundray
Copy link
Contributor

@kevaundray kevaundray commented Jan 26, 2025

Original documents are here (for validators):

@github-actions github-actions bot added c-new Creates a brand new proposal s-draft This EIP is a Draft t-informational labels Jan 26, 2025
@eth-bot
Copy link
Collaborator

eth-bot commented Jan 26, 2025

✅ All reviewers have approved.

@eth-bot eth-bot added the e-consensus Waiting on editor consensus label Jan 26, 2025
Comment on lines 97 to 103
- Statista states that as of January 2024:
- The global average download for broadband is 92 Mbps and the global average upload is 43 Mpbs.
- The global average download for mobile is 50 Mbps and 11 Mbps
- GSMA report showing the state of mobile internet connectivity in 2024 shows that:
- The mobile upload speeds in Higher Income Countries (HIC) is about 18 Mbps
- The global average mobile download is 48 Mbps.

Copy link
Contributor Author

@kevaundray kevaundray Jan 26, 2025

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Let me know how you would want these to be referred to in the document

@github-actions github-actions bot added the w-ci Waiting on CI to pass label Jan 26, 2025

RAM/memory is dominated by state cache. As of January 2025, it is possible to run a full node with 16GB of RAM, however this has been known to not work with all combinations of EL and CL clients in the past.

On 32GB vs 64GB; 32GB works right now, however we recommend 64GB as [preliminary benchmarks](https://hackmd.io/@han/bench-hash-in-snark) have shown that zk-STARKS can consume a significant amount of memory and the difference in cost relative to the entire hardware setup for a validator is insignificant.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I saw that EIPs 2926, 2938, 3298, 3416 and 3607 also used hackmd links however the contents of them can easily be changed, so I wonder if there is a rule about using them?

@github-actions github-actions bot added w-ci Waiting on CI to pass and removed w-ci Waiting on CI to pass labels Jan 26, 2025
Copy link
Contributor

@smartprogrammer93 smartprogrammer93 left a comment

Choose a reason for hiding this comment

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

Given EIPs are immutable once they become final, does it make sense to place these requirements in an EIP? I expect requirements will need to be updated in a couple of years. In that case, we will need to create a different EIP amending this one. Then, operators will need to check 2 different EIPs to know what the current requirements are. Seems impractical to me.

@restakeservice
Copy link

good idea!

@kevaundray
Copy link
Contributor Author

Given EIPs are immutable once they become final, does it make sense to place these requirements in an EIP? I expect requirements will need to be updated in a couple of years. In that case, we will need to create a different EIP amending this one. Then, operators will need to check 2 different EIPs to know what the current requirements are. Seems impractical to me.

In the call someone mentioned that, the status of this could be changed to "live" and we modify it in-place instead of creating a new EIP each time. I would defer to the EIP maintainers regarding that and whats possible there.

I agree that creating a new EIP whenever we change specs is undesirable.

@kevaundray kevaundray changed the title Add EIP: Hardware and Bandwidth Requirements for Full Nodes and Validators Add EIP: Hardware and Bandwidth Recommendations for Full Nodes and Validators Jan 27, 2025
@github-actions github-actions bot removed the w-ci Waiting on CI to pass label Jan 27, 2025
@github-actions github-actions bot added the w-ci Waiting on CI to pass label Jan 27, 2025
@kevaundray
Copy link
Contributor Author

General question to this one:

Should we better treat that as "Recommended setup" or "Minimal setup"?

Minimal would be then something we can refer to and adjust to it - having any users on our support lines which will be struggling with weaker setups could be then advised to think of any improvement and referred to this EIP. Recommendation always gives quite a bit of room for complains.

Also having "Minimal supported setup" in place we can adjust any benchmarking tools to it and then make a proper assumptions.

This would be treated as "recommended" -- you can use weaker hardware but they would not be tested/benchmarked against is the thinking. The minimum is somewhat client specific, so I'd be hesitant to add that maintenance burden and it also would likely change more often per hardfork vs recommended due to the headroom.

@g11tech
Copy link
Contributor

g11tech commented Feb 25, 2025

hey @kevaundray , if you could get the linter/bots to pass, we can merge this PR and then you can continually update this basis discussion and feedback

Co-authored-by: Justin Traglia <[email protected]>
@eth-bot eth-bot changed the title Add EIP: Hardware and Bandwidth Recommendations for Validators and Full Nodes Add EIP: Hardware and Bandwidth Recommendations Feb 27, 2025
@github-actions github-actions bot removed the w-ci Waiting on CI to pass label Feb 27, 2025
Copy link

The commit d61b83b (as a parent of 1c30144) contains errors.
Please inspect the Run Summary for details.

@github-actions github-actions bot added the w-ci Waiting on CI to pass label Feb 27, 2025
@github-actions github-actions bot removed the w-ci Waiting on CI to pass label Feb 27, 2025
@kevaundray
Copy link
Contributor Author

hey @kevaundray , if you could get the linter/bots to pass, we can merge this PR and then you can continually update this basis discussion and feedback

Should be mergeable now :)

@kevaundray
Copy link
Contributor Author

@g11tech bump on merging this


### CPU

- **Single vs. Multi-thread Performance**
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe this section could be expanded slightly, it doesn't give a clear rationale to me

Copy link
Contributor

@g11tech g11tech left a comment

Choose a reason for hiding this comment

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

lgtm for draft, whether to anchor in time via title or having it as a living EIP, that call can be taken at moving to review time

@eth-bot eth-bot enabled auto-merge (squash) March 14, 2025 15:19
Copy link
Collaborator

@eth-bot eth-bot left a comment

Choose a reason for hiding this comment

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

All Reviewers Have Approved; Performing Automatic Merge...

@eth-bot eth-bot merged commit 01d0d9d into ethereum:master Mar 14, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c-new Creates a brand new proposal e-consensus Waiting on editor consensus e-review Waiting on editor to review s-draft This EIP is a Draft t-informational
Projects
None yet
Development

Successfully merging this pull request may close these issues.