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

Update operations docs #3883

Merged
merged 19 commits into from May 1, 2024
Merged

Update operations docs #3883

merged 19 commits into from May 1, 2024

Conversation

SEJeff
Copy link
Collaborator

@SEJeff SEJeff commented Apr 16, 2024

This is a set of changes I've been working on as I saw RockawayX folks ask questions when they were working on setting up a new guardian node.

@SEJeff SEJeff marked this pull request as ready for review April 24, 2024 20:23
@SEJeff SEJeff requested a review from barnjamin April 24, 2024 21:17
bruce-riley
bruce-riley previously approved these changes Apr 25, 2024
docs/operations.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@evan-gray evan-gray left a comment

Choose a reason for hiding this comment

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

Overall good and beneficial changes, just some grammar nits. Also there's some inconsistency in the header title casing.

docs/operations.md Outdated Show resolved Hide resolved
docs/operations.md Outdated Show resolved Hide resolved
docs/operations.md Show resolved Hide resolved
docs/operations.md Outdated Show resolved Hide resolved
docs/operations.md Outdated Show resolved Hide resolved
docs/operations.md Outdated Show resolved Hide resolved
docs/operations.md Outdated Show resolved Hide resolved
docs/operations.md Outdated Show resolved Hide resolved
docs/operations.md Outdated Show resolved Hide resolved
docs/operations.md Outdated Show resolved Hide resolved
bruce-riley
bruce-riley previously approved these changes Apr 29, 2024
docs/operations.md Outdated Show resolved Hide resolved
Shoutout to @0xshipthecode and the RockawayX Infra team for helping test
and develop the patch for this in #3786.
This shows new guardians how to enable telemetry.
The lcd was built directly into the terra binary a long time ago and no
longer requires a separate service for the lcd. This happened more than
a year ago.
* mention the wormhole-dashboard github hosted instance
* mention how to run the fly healthcheck to verify a guardian
So that guardians know which nodes they need to run and which they can
rely on wormchain for.
* Since wormhole supports 30+ chains now and not just 5 chains, the
  solana, eth, terra, bsc bits were removed
* The solana and terra content had the urls updated to reflect reality
  and moved into their own sections.
* Made a reference to docs.wormhole.com's constants reference for chains
Also mention threshold signing via horcrux and using the sentry node
architecture to protect the validator from volumetric attacks.
* high level overview of how to enable the NTT Accountant
* emphasize all guardians need to run wormchain validators
* remove the reference to checking out a v2.x branch as the main branch
  is for wormholev2 now and has been for some time.
Try to make things consistent for @evan-gray.
@SEJeff SEJeff dismissed evan-gray’s stale review April 30, 2024 19:33

Evan is happy now

@evan-gray evan-gray merged commit 21e2c87 into main May 1, 2024
24 checks passed
@evan-gray evan-gray deleted the update-operations-docs branch May 1, 2024 03:14
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.

None yet

4 participants