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

docs: ibc relaying and chain upgrades #25

Merged
merged 1 commit into from
Aug 7, 2024
Merged

docs: ibc relaying and chain upgrades #25

merged 1 commit into from
Aug 7, 2024

Conversation

conorsch
Copy link
Contributor

@conorsch conorsch commented Aug 3, 2024

First pass at making sure the necessary steps for updating relayers across upgrade boundaries are well documented.

@@ -28,7 +28,7 @@ planned upgrade. The process then becomes:
1. Restore node state from backup.
2. Ensure you're using the appropriate `pd` and `cometbft` versions for the associated state.
3. Run `pd migrate --ready-to-start` to permit `pd` to start up.
4. Run CometBFT with extra options: `--p2p.pex=false --p2p.seeds='' --moniker archive-node-1`
4. Run CometBFT with extra options: `--p2p.pex=false --p2p.seeds='' --p2p.persistent_peers='' --p2p.laddr="tcp://127.0.0.1:26656" --moniker archive-node-1`
Copy link
Contributor Author

Choose a reason for hiding this comment

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

These options are based on feedback we received from TM/CMT folks on best practices. Trying to be specific, but there's actually a lot one needs to do to make sure a node is not talking to peers, like nuking the address book. Even if the node does peer and disagrees about block contents, it'll still serve its RPC, which means its still suitable for as an archive node, at least for relaying.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added a step to move aside the addrbook, after conferring with some relayer operators about best practices.

Copy link

github-actions bot commented Aug 3, 2024

Visit the preview URL for this PR (updated for commit aea20cb):

https://penumbra-guide--pr25-relayer-upgrades-68xtp1ht.web.app

(expires Wed, 14 Aug 2024 21:54:10 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: c64218fe717b75c9724a23946df883a1adf10faa

Add more detailed descriptions of running an archive node and performing
genesis-restart maintenance for relayers.
@conorsch conorsch requested a review from aubrika August 7, 2024 21:52
@conorsch conorsch marked this pull request as ready for review August 7, 2024 21:52
@conorsch
Copy link
Contributor Author

conorsch commented Aug 7, 2024

Updated based on recent feedback. Ready for review!

Copy link
Contributor

@aubrika aubrika left a comment

Choose a reason for hiding this comment

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

LGTM, great updates 🙏🏻

@aubrika aubrika merged commit 4627f3e into main Aug 7, 2024
2 checks passed
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.

2 participants