Skip to content
This repository has been archived by the owner on Apr 15, 2024. It is now read-only.

docs: document the 5% threshold to change the validator set #331

Merged
merged 3 commits into from
Apr 28, 2023

Conversation

rach-id
Copy link
Member

@rach-id rach-id commented Apr 27, 2023

Overview

Closes #328

Checklist

  • New and updated code has appropriate documentation
  • New and updated code has new and/or updated testing
  • Required CI checks are passing
  • Visual proof for any user facing features like CLI or documentation updates
  • Linked issues closed with keywords

@rach-id rach-id added documentation Improvements or additions to documentation orchestrator orchestrator related labels Apr 27, 2023
@rach-id rach-id self-assigned this Apr 27, 2023
@rach-id rach-id requested a review from evan-forbes as a code owner April 27, 2023 12:23
@rach-id rach-id requested a review from jcstein April 28, 2023 13:40
jcstein
jcstein previously approved these changes Apr 28, 2023
@rach-id rach-id enabled auto-merge (squash) April 28, 2023 13:42
@@ -236,6 +236,8 @@ celestia-appd query staking validator <validator_valoper_address>

Now, you can restart the orchestrator, and it should start signing.

Note: A validator set change is triggered if more than 5% of the total staking power of the network changes (0.5% for BSR). This means that even if you change your EVM address, and you don't see your orchestrator signing, it's alright. Just wait until the validator set changes, and then your orchestrator will automatically start signing.
Copy link
Member

Choose a reason for hiding this comment

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

I might simply mention that this could ber different per network, or explain BSR here since out of context it might be confusing

Copy link
Member Author

Choose a reason for hiding this comment

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

I put here BSR because first validators who will read this will be from BSR. We can update it later. What do you think?

@rach-id rach-id merged commit 82532a0 into celestiaorg:main Apr 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Improvements or additions to documentation orchestrator orchestrator related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Document the 5% staking threshold for orchestrators
3 participants