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

Split Set Up a Validator page in multiple pages and add improve Key Management docs #404

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

0xLucca
Copy link
Collaborator

@0xLucca 0xLucca commented Feb 19, 2025

This PR restructures the Set Up a Validator page by splitting it into three dedicated sections:

  • Setup – covers environment configuration
  • Key Management – Expands on key handling, including subkey usage
  • Start Validating – guides users through launching and maintaining a validator

Additionally, the Key Management section has been improved

@0xLucca 0xLucca requested a review from a team as a code owner February 19, 2025 15:05
@0xLucca 0xLucca added B0 - Needs Review Pull request is ready for review C1 - Medium Medium priority task A1 - Maintenance Major Pull request contains major updates to an existing page (i.e., adding a new section, reorgs, etc.) labels Feb 19, 2025
@0xLucca 0xLucca requested a review from nhussein11 February 19, 2025 15:06
Copy link
Collaborator

@nhussein11 nhussein11 left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -2,5 +2,7 @@ title: Onboarding and Offboarding
nav:
- index.md
# - 'Community Validator Program': community-validator-program.md
- 'Set Up a Validator': set-up-validator.md
- 'Setup': set-up-validator.md
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think this should be "Set Up" (instead of Setup) as the title of the page is "#Set Up a Validator"

Copy link
Collaborator

Choose a reason for hiding this comment

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

Setup should be used if this is more of an overview. Set Up would be more appropriate for a guide. I'd probably keep it as Set Up a Validator because Set Up as a standalone title feels off 🤷‍♀️

Running your validator on a test network like Westend or Kusama is a smart way to familiarize yourself with the process and identify any setup issues in a lower-stakes environment before joining the Polkadot MainNet.

- **Westend** - Polkadot's primary TestNet is open to anyone for testing purposes. Validator slots are intentionally limited to keep the network stable for the Polkadot release process, so it may not support as many validators at any given time
- **Kusama** - often called Polkadot's “canary network,” Kusama has real economic value but operates with a faster and more experimental approach. Running a validator here provides an experience closer to MainNet with the benefit of more frequent validation opportunities with an era time of 6 hours vs 24 hours for Polkadot
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
- **Kusama** - often called Polkadot's canary network, Kusama has real economic value but operates with a faster and more experimental approach. Running a validator here provides an experience closer to MainNet with the benefit of more frequent validation opportunities with an era time of 6 hours vs 24 hours for Polkadot
- **Kusama** - often called Polkadot's "canary network," Kusama has real economic value but operates with a faster and more experimental approach. Running a validator here provides an experience closer to MainNet with the benefit of more frequent validation opportunities with an era time of 6 hours vs 24 hours for Polkadot

Copy link
Collaborator

@eshaben eshaben left a comment

Choose a reason for hiding this comment

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

Love this change. Much needed!

@@ -0,0 +1,126 @@
---
title: Key Management
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
title: Key Management
title: Validator Key Management


### Generate Session Keys

There are multiple ways to create the session keys. It can be done by interacting with the Polkadot.js Apps UI, using the curl command or by using [Subkey](https://paritytech.github.io/polkadot-sdk/master/subkey/index.html){target=\_blank}.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Link to Polkadot.js Apps might be nice

Comment on lines +51 to +53
To properly store these keys, create a file in your keystore directory with a specific naming convention. The filename must consist of two parts: the hex string `61757261` (which represents "aura" in hex) followed by the public key without its `0x` prefix.

The name of the file must be the public key prepended with `61757261` (hex representation of aura) and without the 0x prefix, while the content of the file has to be the secret phrase.
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is duplicated

2. In the **Toolbox** tab, navigate to **RPC calls**
3. Select **`author_rotateKeys`** from the drop-down menu and run the command. This will generate new session keys in your node's keystore and return the result as a hex-encoded string
4. Copy and save this hex-encoded output for the next step

Copy link
Collaborator

Choose a reason for hiding this comment

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

Might be nice to add an image here

@@ -2,5 +2,7 @@ title: Onboarding and Offboarding
nav:
- index.md
# - 'Community Validator Program': community-validator-program.md
- 'Set Up a Validator': set-up-validator.md
- 'Setup': set-up-validator.md
Copy link
Collaborator

Choose a reason for hiding this comment

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

Setup should be used if this is more of an overview. Set Up would be more appropriate for a guide. I'd probably keep it as Set Up a Validator because Set Up as a standalone title feels off 🤷‍♀️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A1 - Maintenance Major Pull request contains major updates to an existing page (i.e., adding a new section, reorgs, etc.) B0 - Needs Review Pull request is ready for review C1 - Medium Medium priority task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants