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] Create documentation for multinode clusters #1820

Closed
zivkovicmilos opened this issue Mar 25, 2024 · 7 comments · Fixed by #2285
Closed

[docs] Create documentation for multinode clusters #1820

zivkovicmilos opened this issue Mar 25, 2024 · 7 comments · Fixed by #2285

Comments

@zivkovicmilos
Copy link
Member

Description

This issue concerns creating official documentation on spinning up multinode clusters that cover:

  • how to generate / fetch genesis.json
  • how to generate / manage node secrets
  • how to generate / manage node config files
  • how to spin up a completely new network
  • how to connect to an existing network
@Kouteki
Copy link
Contributor

Kouteki commented Apr 2, 2024

Blocked by #1821

@zivkovicmilos zivkovicmilos removed their assignment Apr 3, 2024
@adr-sk
Copy link
Contributor

adr-sk commented Apr 15, 2024

As part of Onbloc's multi-node testnet initiative effort, we have tested the newly added gnoland config and gnoland secrets commands to start and connect to a multi-node Gno Chain. Sharing our feedback and suggestions here based on our experience to help improve the Validator management flow: docs

Test Environment:

  • Branch: master
  • Last Commit: aab9f4
  • Validator Node Specs
    • EC2: 2 CPU, 4GB RAM // 1st Validator
    • EC2: 2 CPU, 2GB RAM // 2nd Validator

Stress Test

Performed with the 2nd Validator (2 CPU, 2GB RAM) using Supernova

// Under normal circumstances ( CPU % / RAM GB)
7% / 0.5 GB

// 300 realm call transactions
+7% / +0.02 GB

// 300 realm deployment transactions
+15% / +0.06 GB

Feedback from our Tests

  1. There needs to be a way to get the node-id from the CLI level. [chain] Add a get-node-id flag to gnoland secrets // multi-node related #1922
  2. The unconditional_peer_ids is missing from the config.toml. This is an essential option that is required for setting up the Sentry Architecture to achieve higher security of the Validator Node. [chain] Add unconditional_peer_ids to gnoland configuration #1923
  3. Missing support for pruning. (All nodes are archive nodes for now. Might need to analyze the long-term disk usage.)

With the Gno team's feedback, we’d like to test it further to streamline the multi-node setup experience and help complete the documentation efforts, similar to our past Gnoland Developer Portal initiative.

Please let us know what you think about our suggestions. @zivkovicmilos

(cc @r3v4s @dongwon8247)

@zivkovicmilos
Copy link
Member Author

zivkovicmilos commented Apr 16, 2024

@adr-sk @r3v4s @dongwon8247

Thank you for the feedback on this initial setup, and for opening up the issues 🙏

I'm trying to figure out what the best approach for #1922 is, since for the node's node_id, you need a wombo combo of the derived address from the secret (required secret), as well as the listen address of the node (configuration, not required).
I'm not sure if it should slap on the default node ID if no configuration is present, and read from the actual configuration if it is.

What would be the CLI API or showing this, given how gnoland secrets work?

@r3v4s
Copy link
Contributor

r3v4s commented Apr 16, 2024

Hey @zivkovicmilos, for form of node_id@ip:port, can we just combine these 2 command output?

  • gnoland secrets get NodeKey
  • gnoland config get p2p.laddr

@zivkovicmilos
Copy link
Member Author

Hey @zivkovicmilos, for form of node_id@ip:port, can we just combine these 2 command output?

  • gnoland secrets get NodeKey
  • gnoland config get p2p.laddr

Yes, you are absolutely correct, I'm just not sure what the command chain would be to call this functionality. I'd prefer to keep gnoland secrets, and gnoland config with the current API, and figure out another solution for this UX feature, maybe another command suite that just combines other commands, ex gnoland meta

What do you think?

cc @gfanton @ajnavarro @thehowl

@r3v4s
Copy link
Contributor

r3v4s commented Apr 30, 2024

@zivkovicmilos How about gnoland validator or gnoland node?

I think gnoland meta should provide something like gnoland meta version(daemon or binary version)

@r3v4s
Copy link
Contributor

r3v4s commented May 10, 2024

// UPDATE

Missing support for pruning. (All nodes are archive nodes for now. Might need to analyze the long-term disk usage.)

Sorry for wrong info. Currently when we start gnoland chain it doesn't archive but prune everything.

@r3v4s r3v4s self-assigned this May 10, 2024
gfanton pushed a commit to gfanton/gno that referenced this issue Jul 23, 2024
<!-- please provide a detailed description of the changes made in this
pull request. -->

This PR adds the `validator` docs and updates old `gnoland` cli docs.
Closes gnolang#1820 

<details><summary>Contributors' checklist...</summary>

- [x] Added new tests, or not needed, or not feasible
- [x] Provided an example (e.g. screenshot) to aid review or the PR is
self-explanatory
- [x] Updated the official documentation or not needed
- [x] No breaking changes were made, or a `BREAKING CHANGE: xxx` message
was included in the description
- [x] Added references to related issues and PRs
- [ ] Provided any useful hints for running manual tests
- [ ] Added new benchmarks to [generated
graphs](https://gnoland.github.io/benchmarks), if any. More info
[here](https://github.com/gnolang/gno/blob/master/.benchmarks/README.md).
</details>

---------

Co-authored-by: adr-sk <[email protected]>
Co-authored-by: Milos Zivkovic <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants