docs(mesa): add ADX to the hardware table, correct the seed libp2p keypair guidance - #1219
Merged
Merged
Conversation
…uidance Source: mesa-tracker-fix-implementation_v3.txt, Table E. Companion to MinaProtocol/mesa-upgrade-status#37, which fixes the same hardware string on the upgrade tracker. The tracker copied its wording from this page, so A1 there is only half the fix without E1 here. E1 — the CPU instruction set list was incomplete. Mesa needs BMI2, ADX and AVX. Corrected in all three places: the Mina Daemon Node row, the SNARK Worker row, and the "All operators" checklist line, which also used the "AVX/BMI2" short form. E4 — the libp2p guidance said that all seed nodes must start with the *same* keypair. That is wrong. A libp2p keypair determines the peer ID of the node, so seed nodes cannot share one. The published seed lists confirm this: the three devnet seeds and all eleven mainnet seeds each carry a distinct peer ID in bootnodes.minaprotocol.com. The dedicated page, node-operators/seed-peers/generating-a-libp2p-keypair, already says "stable", not "same". This page now agrees with it: each seed node needs its own keypair, and that keypair must not change across restarts. E3 — partial. The zkApp checklist said only that every zkApp must be redeployed, which reads as though on-chain state is lost. It is not. network-upgrades/mesa/upgrade-steps/examples.mdx already states that the account and its state fields at indexes 0-7 carry over, and that only the Berkeley verification key becomes invalid. That fact is now on the checklist too. Whether a redeploy is required at all, or whether a verification key update is enough, is still an open question for the o1js owner; this change does not answer it. Not in this change: - E2, the storage and memory figures. This page says 16 GB storage for the daemon and the SNARK coordinator and 1 GB for the SNARK worker, and 8 GB RAM per SNARK worker. The upgrade blog and the devnet operator document give 64 GB and ~4 GB per SNARK process. I have no way to tell which side is right, and wrong hardware figures on an operator page are worse than none. This needs the docs owner. - E5, the libp2p flag. No change is needed. This page uses --privkey-path, which matches the flag declared in mina src/lib/cli_lib/flag.ml:22. - network-upgrades/berkeley/requirements.mdx carries the same wrong "same libp2p keypair" sentence. It is a historical upgrade page, so it is left for the docs owner to decide. static/llms-full.txt regenerated, as scripts/check-llms-txt requires. Verified with a full docusaurus build. The broken anchor warnings are pre-existing and come from the mesa index page, not from this change. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
yamimaio
approved these changes
Aug 25, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Table E of
mesa-tracker-fix-implementation_v3.txt(25 Aug 2026). Companion to MinaProtocol/mesa-upgrade-status#37, which fixes the same hardware string on the upgrade tracker. The tracker copied its wording from this page, so the tracker fix (A1) is only half the correction without E1 here.All of it is on
docs/network-upgrades/mesa/requirements.mdx.E1 — the CPU instruction set list is incomplete
Mesa needs BMI2, ADX and AVX.
ADXwas missing in all three places:AVX/BMI2short formE4 — seed nodes must not share a libp2p keypair
The page said that all seed nodes must start with the same
libp2pkeypair. That is wrong. A libp2p keypair determines the peer ID of the node, so seed nodes cannot share one.The published seed lists confirm it — every entry carries a distinct peer ID:
The dedicated page,
node-operators/seed-peers/generating-a-libp2p-keypair, already says stable, not same. This page now agrees with it: each seed node needs its own keypair, and that keypair must not change across restarts.E3 — partial
The zkApp checklist said only that every zkApp must be redeployed, which reads as though on-chain state is lost. It is not.
network-upgrades/mesa/upgrade-steps/examples.mdxalready states that the account and its state fields at indexes0-7carry over, and that only the Berkeley verification key becomes invalid. That fact is now on the checklist too.This does not close E3. Whether a redeploy is required at all, or whether a verification key update is enough, is still an open question for the o1js owner.
Not in this PR
--privkey-path, which matches the flag declared atsrc/lib/cli_lib/flag.ml:22in the mina repo. The single-dash-privkey-pathform is the one in the source docs, not here.Also noted:
docs/network-upgrades/berkeley/requirements.mdx:59carries the same wrong "same libp2p keypair" sentence. It is a historical upgrade page, so I left it for the docs owner to decide.Verification
static/llms-full.txtregenerated, asnpm run check-llms-txtrequires.static/llms.txtis unchanged, because no frontmatter changeddocusaurus buildpasses. The broken anchor warnings come from the mesa index page and are pre-existingAVX/BMI2no longer appears anywhere underdocs/orstatic/🤖 Generated with Claude Code