gaspersic/solana
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Drop the recommendation that --expected-shred-version be set by validators

Browse Source 
`--expected-shred-version` is another knob for users to get wrong and is
documentation that can get stale due to cluster restarts.  Turns out
it's also generally not required anymore either because:
1. The cluster entrypoint can always be expected to be using the correct
   shred version, and that shred version will be adopted by the new node
   (earlier this was not the case when the `solana-gossip spy` node on
   mainnet-beta.solana.com:8001 ran with shred version 0)
2. On a cluster restart, `--expected-bank-hash` is a much stronger
   assertion that the validator is starting from the correct place (and
   didn't exist when `--expected-shred-version` was first recommended)
This commit is contained in:
Michael Vines
2020-09-15 09:48:23 -07:00
parent b85e8497b5
commit 4ada4d43f2
3 changed files with 1 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
docs/src/running-validator/restart-cluster.md
View File

@@ -21,7 +21,6 @@ call this NEW\_SHRED\_VERSION and NEW\_BANK\_HASH respectively.
Adjust your validator's arugments:
```bash
--expected-shred-version NEW_SHRED_VERSION
--wait-for-supermajority SLOT_X
--expected-bank-hash NEW_BANK_HASH
```
@@ -51,7 +50,6 @@ Post something like the following to #announcements (adjusting the text as appro
> ```bash
> solana-validator
> --wait-for-supermajority 12961040 # <-- NEW! IMPORTANT!
> --expected-shred-version 56096 # <-- NEW! IMPORTANT!
> --expected-bank-hash 6q2oTgs8FiJxra2Zo1N8tzWqo5b6uGbjmFgoWDsXxchY # <-- NEW! IMPORTANT!
> --hard-fork 56096 # <-- NEW! IMPORTANT!
> --entrypoint 35.203.170.30:8001 # <-- Same as before