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

Display up-to-date node storage size #809

Merged
merged 21 commits into from
Nov 8, 2024
Merged

Display up-to-date node storage size #809

merged 21 commits into from
Nov 8, 2024

Conversation

jlwllmr
Copy link
Collaborator

@jlwllmr jlwllmr commented Nov 4, 2024

Adds functionality to update node volume size values weekly.

  1. The script fetchNodeSize.js is the main element. It sends the request to get the data, then writes the data to linea-node-size/data.json. It retrieves data for the nodes specified in /linea-node-size/config.json, which can be easily updated/modified if we want to use other nodes as sources.
  2. Two React components then handle the implementation in the MDX files. NodeSize.jsx displays the values, while LastUpdated.jsx provides a timestamp. Both use the data.json as their source.
  3. A Github Actions workflow will run weekly (00:00 on Mondays, UTC) and create a PR with the changes to the data.json file. Once merged, the node values will be refreshed. data.json should be the only item that requires updates, as the components will always pull the most recent data from the JSON.

image

Currently, the components are not present in the Erigon guide, as we need to establish to what extent this differs from Geth in terms of storage requirements. The Besu and Linea Besu guides use the same figures, as these should be identical in practice.

Copy link

vercel bot commented Nov 4, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
doc-zk-evm ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 8, 2024 9:49am

@mapachurro
Copy link
Collaborator

Hmmm
image
Looking into this

Copy link
Collaborator

@mapachurro mapachurro left a comment

Choose a reason for hiding this comment

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

I love this thing!

Happy to help you tweak the GHA if they don't work right out of the gate. 🙏

@jlwllmr
Copy link
Collaborator Author

jlwllmr commented Nov 8, 2024

Attempting to fix the fact the linter won't run. It seems we are somehow in a situation where:

  • eslint >8.57.0 is not supported, so we should transition to eslint ^9.0.0. But that requires several additional packages
  • These additional packages cause dependency issues with other eslint-related packages, when installed
  • We already switched to the modern config format in Bump dependencies and edit linter config formats #708
  • So setting up our repo for the linter to work would mean reverting to the older config format, and then awaiting all the plugins to catch up with eslint ^9.0.0 and become compatible

Going to merge anyway, may fix in a separate PR.

@jlwllmr jlwllmr merged commit d5f02a2 into main Nov 8, 2024
8 of 10 checks passed
@jlwllmr jlwllmr deleted the node-size-2 branch November 8, 2024 09:52
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