Cover linux/arm64 in the prebuilt seid install and upgrade steps - #64
Draft
monty-sei wants to merge 1 commit into
Draft
Cover linux/arm64 in the prebuilt seid install and upgrade steps#64monty-sei wants to merge 1 commit into
monty-sei wants to merge 1 commit into
Conversation
The prebuilt binary tabs hardcode the amd64 tarball name and state that the builds are amd64 only. Once arm64 binaries are attached, an operator on Graviton or Ampere following these pages downloads a 404 or, worse, installs an amd64 binary that will not run. Derive the asset architecture from uname -m rather than substituting it directly. The asset token is arm64 while uname -m reports aarch64 on Linux, so a naive substitution works on amd64 by coincidence and fails everywhere else. An unrecognised architecture now stops the pipeline instead of building a filename that does not exist. Both update procedures in node-operators.mdx get the same treatment. They were easy to miss and leaving them would let arm64 operators install and then be unable to upgrade.
Contributor
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
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
1 participant
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.
Draft on purpose. Do not merge until arm64 binaries are actually attached to a sei-chain release. These pages were reverted once before (#44) for documenting prebuilt binaries ahead of their existence, and #55 only restored them after v6.6.1 shipped. Same rule applies here.
What this changes
Three files, eight lines. The prebuilt binary tabs hardcode
sei-chain_${VERSION#v}_linux_x86_64.tar.gzand state the builds arelinux/amd64only.evm/installing-seid-cli.mdx<Info>caveatnode/index.mdx<Info>caveatnode/node-operators.mdxThe two in
node-operators.mdxare the easy ones to miss. Without them an arm64 operator can install from the guide and then be unable to upgrade, because both update procedures would hand them an amd64 tarball.The detail worth reviewing
The asset token is
arm64, butuname -mon Linux reportsaarch64. So substituting$(uname -m)directly appears to work (it is correct on amd64 by coincidence) and then 404s on every arm64 machine. The snippet maps explicitly instead:An unrecognised architecture stops rather than building a filename that cannot exist. Verified the mapping produces
..._linux_x86_64.tar.gzonx86_64and..._linux_arm64.tar.gzon bothaarch64andarm64.Before this can merge
<arm64-release-tag>placeholder inevm/installing-seid-cli.mdxandnode/index.mdxwith the first release that carries themsei-chain_<version>_linux_arm64.tar.gz)checksums.txt, sincesha256sum -c --ignore-missingexits 0 and prints nothing for a file that is present but unlistedOut of scope
node/seictl.mdxalso names x86_64 archives, but that is the seictl binary and a separate release pipeline.node/index.mdxline 195 ("Available architectures: linux/amd64 and linux/arm64") describes the Docker images and stays accurate.llms.txtandllms-full.txtare generated from the deployed site, so they should be regenerated after this deploys rather than hand-edited.