docs: Node operator doc fixes #1771
Merged
Conversation
This file contains 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
PhilippGackstatter
added
the
sc-platform
fijter
approved these changes
Aug 15, 2024
lucas-tortora
approved these changes
Aug 15, 2024
PhilippGackstatter
force-pushed
the
vm-language/feat/operator-docs-updates
branch
from
7f8a1b5 to
0ebcc59
Compare
|
✅ Vercel Preview Deployment is ready! |
|
✅ Vercel Preview Deployment is ready! |
|
This PR is ready to be merged but for some reason the |
|
✅ Vercel Preview Deployment is ready! |
|
@PhilippGackstatter repo settings need to be updated to require the correct job, I asked Thomas to update |
|
✅ Vercel Preview Deployment is ready! |
Thoralf-M
reviewed
Aug 16, 2024
bingyanglin
pushed a commit
that referenced
this pull request
Aug 19, 2024
* fix(docs): Replace "a IOTA" with "an IOTA" * fix(docs): Typos and incorrect URLs in full node config * fix(docs): Update staking numbers and fix style * fix(nre): Fix typos, style and urls in node operator guide * fix(docs): Fix data management typos and content * fix(docs): Typos and style fixes in snapshots * feat(devx): poc import config READMEs in wiki while keeping the READMEs valid * fix(docs): Proofread node tools * fix(docs): Rename tasks section to operation * fix(docs): Proofread node monitoring * fix(docs): Comment out TODO URLs * fix(docs): dprint fmt * refactor(docs): Pull validator tool into wiki as well --------- Co-authored-by: Lucas Tortora <[email protected]> Co-authored-by: Thoralf-M <[email protected]>
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.
Description of change
Proofread the node operator section. Deduplicates the validator task file which existed in the wiki and in the
nredirectory within the repo (thanks @lucas-tortora).A local self-compiled network was successfully setup following the instructions in the wiki pages.
The docker, ansible and systemd setup was NOT tested. It seems much more appropriate for someone from DevOps to check these since they have the most experience with node deployment and operation. They can also better decide whether we want to support all of those configurations or perhaps only the docker one, for example.
Tasks that weren't possible to do in this PR or follow-up tasks from this PR are documented in #1867.
Links to any relevant issues
fixes #1290