Update assumed chain params

sriramdvt commented at 2:17 pm on July 19, 2021: contributor

Update the relevant variables in src/chainparams.cpp for mainnet, testnet, and signet as given here.

To review this PR, check out this guide.

Note: added a 10% overhead to the base value of mainnet in m_assumed_blockchain_size

Note: 10% overhead to the base value of `mainnet` in `m_assumed_blockchain_size`

eeddd1c8fa

MarcoFalke added this to the milestone 22.0 on Jul 19, 2021

DrahtBot added the label Validation on Jul 19, 2021

DrahtBot commented at 6:52 pm on July 19, 2021: member

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Conflicts

Reviewers, this pull request conflicts with the following ones:

#21702 by JeremyRubin

If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

MarcoFalke commented at 7:21 pm on July 19, 2021: member

ACK eeddd1c8fa96cf546b0bf92063cefa4fd8c6b415, checked against my node 🌮

Signature:

 0-----BEGIN PGP SIGNED MESSAGE-----
 1Hash: SHA512
 2
 3ACK eeddd1c8fa96cf546b0bf92063cefa4fd8c6b415, checked against my node 🌮
 4-----BEGIN PGP SIGNATURE-----
 5
 6iQGzBAEBCgAdFiEE+rVPoUahrI9sLGYTzit1aX5ppUgFAlwqrYAACgkQzit1aX5p
 7pUio0Av/T1U3uYofh+177Fxc7FtIiv+Am/nWVIc9dJCZuLH/JD+5dbuRmPJjbPf2
 88oQOvABSmIdiUlAC1En9NN7fcwhVnYszzIyJj6ZddstTfJ2amMgzBOJQBFFCG6Jr
 9Np1bpdYC0DOjJuLjUMFIumBZ9v+zGn9jjMwN2jPbZZmVgqxnmY31b1WGFq/cp0Kb
10IfHCX7s+tlc3XE85Pz8xevrzQG7sW4mFYaunTVeOMrF1O1Aiyea5rguO3w/0rFQC
11brJIPjeNDKBDA5N4EpKrXEJg5LvzLDFtV4aZlvlMcf0DelzblGwLKkuIPtsr0bwm
12cNhDNF3d/PnMbw+78eU7xo1Yd+10/MPeHqkzess/4w49ZsvgNPb6MC1gZqVQCrUg
13nfYR9pOobli3LOQTJJLAo1bivil5Oi0t0GS0eGUhNC1hGlIYek+TdpV1lprGvlvf
14HlqFLJRUQmkAbtNENu7IDIP/z4cMSCvARIgAXzYoEN5Yaq3mp571GQjM5agYkAMV
15swQyDggZ
16=JygT
17-----END PGP SIGNATURE-----

Timestamp of file with hash 3c7e2e6989616ff21b62fc11616445232fa5732e9be16947454f63900bce377d -

MarcoFalke commented at 7:27 pm on July 19, 2021: member

I checked all networks, to clarify. The unchanged file sizes were still up-to-date.

jamesob approved

jamesob commented at 7:58 pm on July 19, 2021: member

ACK eeddd1c8fa96cf546b0bf92063cefa4fd8c6b415 (jamesob/ackr/22499.1.sriramdvt.update_assumed_chain_par)

Built locally, verified CMainParams value changes (but not regtest) using getchaintxstats/getblockheader for block 00000000000000000008a89e854d57e5667df88f1cdef6fde2fbca1de5b639ad on my node.

 0-----BEGIN PGP SIGNED MESSAGE-----
 1Hash: SHA512
 2
 3ACK eeddd1c8fa96cf546b0bf92063cefa4fd8c6b415 ([`jamesob/ackr/22499.1.sriramdvt.update_assumed_chain_par`](https://github.com/jamesob/bitcoin/tree/ackr/22499.1.sriramdvt.update_assumed_chain_par))
 4
 5Built locally, verified `CMainParams` value changes (but not regtest) using 
 6`getchaintxstats`/`getblockheader` for block 
 7`00000000000000000008a89e854d57e5667df88f1cdef6fde2fbca1de5b639ad` on my node.
 8-----BEGIN PGP SIGNATURE-----
 9
10iQIzBAEBCgAdFiEEGNRVI1NPYuZCSIrGepNdrbLETwUFAmD12TEACgkQepNdrbLE
11TwXGFQ//XYIo/NYpm70jdXp8oHncRFoahF0fb7M0mDRdlemYfaNs0rv+czuRM6oK
12/1uCyFI5uveAQ1XxvFhxYqyvz+xvzO1ifhj+N/6Zj/xWGWcLmJaPo6quXNam17PB
133IszwgNPLOUbtmYzwDaA0x2/mSpBGlDW5cXoflOClEAVSzasrUTvSHWyTQFTO2oJ
14byipzYF2yN7M+OPpHRSq8XwrcigyzY6LYLmg2xT9+JekH2p0fuPVlgSiJF+wvAzA
15yRt5dQGR0nhRgmUZOSlYRCz0XPcBO6yq0s3PBFh31HYwXgPog4fEI+e1f4tQN4Q+
16Ux/5ydbDKiiwgjwNrt6BKJXyPgXWZmhPjBNaAA3ygqBKSsURbaiZK0Ywq90CKWfx
17OA548wCUJafeSqrfwaszNzvelpynSr8O4kZWs7wdySN5w9U6VNqBF0nYuNo6E5CY
18GaC8GmFUMTTMV+oHkH+yJQLKnMdI7YvXnKbrcT43uOfonNd+70PnSjJLNfIB/PJj
19XBeTh/sokbiDE0OOhq3nVX0iwrmtM6VpyrHK1wtFKfo0j3bjDR8ksqdgVQvrmJTe
207TFhh37We9mR2ZdTH6w2UH0vobArFEfcPIisoKAoWK4OvCAA6LMoEo8jIM68zKYN
21RAx1LgzXIYdc4iRu56PZ83iHBfLofnUgD0hRalo9YjUS7X4guGo=
22=u36D
23-----END PGP SIGNATURE-----

0Tested on Linux-4.19.0-17-amd64-x86_64-with-glibc2.28
1
2Configured with ./configure LDFLAGS=-L/home/james/src/bitcoin2/db4/lib/ CPPFLAGS=-I/home/james/src/bitcoin2/db4/include/ CXXFLAGS=-fPIE -pipe -O2 -g -Wthread-safety-analysis -Wall -Werror=sign-compare -Wsign-compare -Werror=thread-safety-analysis --enable-wallet --enable-debug --with-daemon --enable-natpmp-default CXX=/usr/local/bin/clang++ CC=/usr/local/bin/clang PYTHONPATH= --disable-shared --with-pic --enable-benchmark=no --enable-module-recovery --enable-module-schnorrsig --enable-experimental --disable-shared --with-pic --enable-benchmark=no --enable-module-recovery --enable-module-schnorrsig --enable-experimental --no-create --no-recursion
3
4Compiled with /usr/bin/ccache /usr/local/bin/clang++ -std=c++17 -mavx -mavx2 -fPIE -pipe -O2 -g -Wthread-safety-analysis -Wall -Werror=sign-compare -Wsign-compare -Werror=thread-safety-analysis -O0 -g3 -ftrapv -fdebug-prefix-map=$(abs_srcdir)=.  -Wstack-protector -fstack-protector-all -fcf-protection=full -fstack-clash-protection -msse4 -msha -msse4.1 -msse4.2  i
5
6Compiler version: Debian clang version 11.1.0-++20210428103820+1fdec59bffc1-1~exp1~20210428204437.162

FuzzWorley approved

FuzzWorley commented at 10:56 pm on July 19, 2021: none

I verified CMainParams value changes according to this guide using block header 00000000000000000008a89e854d57e5667df88f1cdef6fde2fbca1de5b639ad on my node.

I did not verify testnet or signet.

 0$ bitcoin-cli getchaintxstats 4096 00000000000000000008a89e854d57e5667df88f1cdef6fde2fbca1de5b639ad
 1{
 2  "time": 1626697539,
 3  "txcount": 656509474,
 4  "window_final_block_hash": "00000000000000000008a89e854d57e5667df88f1cdef6fde2fbca1de5b639ad",
 5  "window_final_block_height": 691719,
 6  "window_block_count": 4096,
 7  "window_tx_count": 7217053,
 8  "window_interval": 2976202,
 9  "txrate": 2.424920418708139
10}

 0$ bitcoin-cli getblockheader 00000000000000000008a89e854d57e5667df88f1cdef6fde2fbca1de5b639ad
 1{
 2  "hash": "00000000000000000008a89e854d57e5667df88f1cdef6fde2fbca1de5b639ad",
 3  "confirmations": 73,
 4  "height": 691719,
 5  "version": 536870916,
 6  "versionHex": "20000004",
 7  "merkleroot": "9ba2c77c80e141c76ea15d9f38973d729eb2d437d9f75f986e50a502ce609b69",
 8  "time": 1626697539,
 9  "mediantime": 1626694708,
10  "nonce": 1396531224,
11  "bits": "17149624",
12  "difficulty": 13672594272814.14,
13  "chainwork": "00000000000000000000000000000000000000001fa4663bbbe19f82de910280",
14  "nTx": 133,
15  "previousblockhash": "0000000000000000000173ead0a321a58ab36b33b0c491283ef6bc09ee47fe49",
16  "nextblockhash": "0000000000000000000070d93e17d3659cda3daa72c2234e628c45569d2476c0"
17}

bfolkens commented at 11:06 pm on July 19, 2021: none

ACK eeddd1c - checked against mainnet

0$ bitcoin-cli getchaintxstats 4096 00000000000000000008a89e854d57e5667df88f1cdef6fde2fbca1de5b639ad | jq '{time,txcount,txrate}'
1{
2  "time": 1626697539,
3  "txcount": 656509474,
4  "txrate": 2.424920418708139
5}

0$ bitcoin-cli getblockheader 00000000000000000008a89e854d57e5667df88f1cdef6fde2fbca1de5b639ad | jq '{hash, height,chainwork}'
1{
2  "hash": "00000000000000000008a89e854d57e5667df88f1cdef6fde2fbca1de5b639ad",
3  "height": 691719,
4  "chainwork": "00000000000000000000000000000000000000001fa4663bbbe19f82de910280"
5}

achow101 commented at 11:30 pm on July 19, 2021: member

Code Review ACK eeddd1c8fa96cf546b0bf92063cefa4fd8c6b415

Checked against my node.

darosior approved

darosior commented at 7:44 am on July 20, 2021: member

ACK eeddd1c8fa96cf546b0bf92063cefa4fd8c6b415 mainnet and testnet

0xB10C commented at 10:15 am on July 20, 2021: member

ACK mainnet, testnet, and signet eeddd1c8fa96cf546b0bf92063cefa4fd8c6b415

Did not check the updated m_assumed_blockchain_size and did not run with reindex-chainstate & assumevalid=0 as mentioned in https://github.com/bitcoin/bitcoin/blob/master/doc/release-process.md#before-branch-off:

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.

MarcoFalke commented at 12:57 pm on July 20, 2021: member

assumevalid=0

Assuming that the previous assumevalid was valid, then just syncing to the tip should be sufficient. Obviously running assumevalid=0 is still useful to catch rejection of past blocks, but this seems unrelated to bumping the assumevalid value.

fanquake merged this on Jul 20, 2021

fanquake closed this on Jul 20, 2021

theStack commented at 3:34 pm on July 20, 2021: member

post-merge ACK eeddd1c8fa96cf546b0bf92063cefa4fd8c6b415 ⛓️ checked updated parameters for mainnet, testnet and signet (modulo m_assumed_blockchain_size, as I have currently pruning enabled)

CommanderPoe commented at 11:45 am on July 23, 2021: none

post-merge ACK mainnet eeddd1c

 0admin@raspberrypi:~ $ bitcoin-cli getchaintxstats 4096 00000000000000000008a89e854d57e5667df88f1cdef6fde2fbca1de5b639ad | jq '{time,txcount,txrate}'
 1{
 2  "time": 1626697539,
 3  "txcount": 656509474,
 4  "txrate": 2.424920418708139
 5}
 6
 7admin@raspberrypi:~ $ bitcoin-cli getblockheader 00000000000000000008a89e854d57e5667df88f1cdef6fde2fbca1de5b639ad | jq '{hash, height,chainwork}'
 8{
 9  "hash": "00000000000000000008a89e854d57e5667df88f1cdef6fde2fbca1de5b639ad",
10  "height": 691719,
11  "chainwork": "00000000000000000000000000000000000000001fa4663bbbe19f82de910280"
12}

gwillen referenced this in commit 79a9c9e715 on Jun 1, 2022

DrahtBot locked this on Aug 18, 2022

Update assumed chain params #22499

Conflicts