Changeset View
Standalone View
doc/release-process.md
Show All 21 Lines | - (major releases) Update [`BLOCK_CHAIN_SIZE`](/src/qt/intro.cpp) to the current size plus | ||||
some overhead. | some overhead. | ||||
- Update `src/chainparams.cpp` defaultAssumeValid and nMinimumChainWork with information from | - Update `src/chainparams.cpp` defaultAssumeValid and nMinimumChainWork with information from | ||||
the getblockhash rpc. | the getblockhash rpc. | ||||
- The selected value must not be orphaned so it may be useful to set the value two blocks back | - The selected value must not be orphaned so it may be useful to set the value two blocks back | ||||
from the tip. | from the tip. | ||||
- Testnet should be set some tens of thousands back from the tip due to reorgs there. | - Testnet should be set some tens of thousands back from the tip due to reorgs there. | ||||
- This update should be reviewed with a reindex-chainstate with assumevalid=0 to catch any defect | - This update should be reviewed with a reindex-chainstate with assumevalid=0 to catch any defect | ||||
that causes rejection of blocks in the past history. | that causes rejection of blocks in the past history. | ||||
- Use `getchaintxstats` rpc to verify statistics about the transaction count and rate. | |||||
Fabien: I think you must backport PR12309 first, this will add some context to this sentence: https… | |||||
jasonbcoxUnsubmitted Not Done Inline ActionsThis also needs to be indented since it's in the chainparams section. jasonbcox: This also needs to be indented since it's in the chainparams section. | |||||
nakihitoAuthorUnsubmitted Not Done Inline Actions@Fabien I actually reworded the line as it was in this PR to better reflect our update process. As far as I'm aware, we don't update transaction rate or count in chainparams.cpp which makes PR12309's first line irrelevant. I didn't include the second line change in 12309 for similar reasons. @jasonbcox This line should be indented to be in line with line 29? Should the next line (31) be indented past that or the same? nakihito: @Fabien I actually reworded the line as it was in this PR to better reflect our update process. | |||||
jasonbcoxUnsubmitted Not Done Inline ActionsNvm. The above block only refers to updating chainparams' assume valid and chain work. It's fine as-is. jasonbcox: Nvm. The above block only refers to updating chainparams' assume valid and chain work. It's… | |||||
jasonbcoxUnsubmitted Not Done Inline ActionsThe more I look into this, the less valuable it looks to me. The only use of chainTxData is for guessing verification progress in the wallet. @deadalnix do you see value in bloating our release process to add updating chainTxData? I don't. jasonbcox: The more I look into this, the less valuable it looks to me. The only use of chainTxData is for… | |||||
- Verify by running `getchaintxstats <window_block_count> <window_last_block_hash>` with the `window_block_count` and `window_last_block_hash` from your output. | |||||
- Regenerate manpages (run `contrib/devtools/gen-manpages.sh`, or for out-of-tree builds run | - Regenerate manpages (run `contrib/devtools/gen-manpages.sh`, or for out-of-tree builds run | ||||
`BUILDDIR=$PWD/build contrib/devtools/gen-manpages.sh`). | `BUILDDIR=$PWD/build contrib/devtools/gen-manpages.sh`). | ||||
- Update seeds as per [contrib/seeds/README.md](/contrib/seeds/README.md) | - Update seeds as per [contrib/seeds/README.md](/contrib/seeds/README.md) | ||||
- Manually test bitcoin-qt by sending some transactions and navigating through the menus. | - Manually test bitcoin-qt by sending some transactions and navigating through the menus. | ||||
4. Add git tag for release | 4. Add git tag for release | ||||
## Release | ## Release | ||||
Show All 32 Lines |
I think you must backport PR12309 first, this will add some context to this sentence: https://github.com/bitcoin/bitcoin/pull/12309