Update stellar-archivist to the new Rust version - #2776
Conversation
|
Preview is available here: |
There was a problem hiding this comment.
Pull request overview
Updates validator documentation from the legacy Go archivist to the Rust implementation.
Changes:
- Documents Rust CLI installation and archive operations.
- Adds scan, repair, mirror, and JSON report examples.
- Updates related links and package listings.
Recommendation: NEEDS-CHANGES — correct the CLI names/flags and the backfill workflow.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
running-node.mdx |
Links to mirror guidance. |
publishing-history-archives.mdx |
Replaces legacy archivist instructions. |
prerequisites.mdx |
Links backfill guidance. |
installation.mdx |
Lists Rust and deprecated Go packages. |
node-operator-tools.mdx |
Adds the Rust archivist. |
💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
60ec381 to
19c9efa
Compare
|
Preview is available here: |
1 similar comment
|
Preview is available here: |
SirTyson
left a comment
SirTyson
left a comment
There was a problem hiding this comment.
I think we might want to flip the framing in the repair section. IMO archives are a hot mess right now, and we want to encourage the simplest thing possible. I think we should lead with the non-plan variant (as it's a one shot command), instead of leaving the non-plan variant as a footnote.
anupsdf
left a comment
anupsdf
left a comment
There was a problem hiding this comment.
Thanks! Posted a few questions.
anupsdf
left a comment
anupsdf
left a comment
There was a problem hiding this comment.
Would be nice to simplify the commands in a followup PR to make it easier for operators.
Yeah, I think that makes sense. I've restructured the repair section to lead with the non-plan variant.
This should also be better now with the above change. |
|
Preview is available here: |
to be merged after stellar/rs-stellar-archivist#32