Page MenuHomePhabricator

[net] Signal NODE_COMPACT_FILTERS if we're serving compact filters.
ClosedPublic

Authored by Fabien on Dec 21 2020, 15:30.

Details

Summary
If -peerblockfilters is configured, signal the NODE_COMPACT_FILTERS
service bit to indicate that we are able to serve compact block filters,
headers and checkpoints.

Partial backport (2/3) of core PR19070:
https://github.com/bitcoin/bitcoin/pull/19070/commits/132b30d9c84f2a8053714a438f227b583a89a9ea

Depends on D8730.

Test Plan
ninja all check-all

Event Timeline

Fabien requested review of this revision.Dec 21 2020, 15:30

Tail of the build log:

[372/431] Running utility command for check-bitcoin-sync_tests
[373/431] bitcoin: testing sigencoding_tests
[374/431] Running utility command for check-bitcoin-bip32_tests
[375/431] Running utility command for check-bitcoin-sigencoding_tests
[376/431] bitcoin: testing torcontrol_tests
[377/431] bitcoin: testing wallet_crypto_tests
[378/431] Running utility command for check-bitcoin-torcontrol_tests
[379/431] Running utility command for check-bitcoin-wallet_crypto_tests
[380/431] bitcoin: testing timedata_tests
[381/431] bitcoin: testing streams_tests
[382/431] Running utility command for check-bitcoin-timedata_tests
[383/431] bitcoin: testing uint256_tests
[384/431] Running utility command for check-bitcoin-streams_tests
[385/431] bitcoin: testing undo_tests
[386/431] bitcoin: testing util_threadnames_tests
[387/431] Running utility command for check-bitcoin-undo_tests
[388/431] Running utility command for check-bitcoin-uint256_tests
[389/431] Running utility command for check-bitcoin-util_threadnames_tests
[390/431] bitcoin: testing validation_chainstatemanager_tests
[391/431] bitcoin: testing compilerbug_tests
[392/431] Running utility command for check-bitcoin-validation_chainstatemanager_tests
[393/431] bitcoin: testing validationinterface_tests
[394/431] bitcoin: testing checkpoints_tests
[395/431] Running utility command for check-bitcoin-validationinterface_tests
[396/431] bitcoin: testing cuckoocache_tests
[397/431] Running utility command for check-bitcoin-compilerbug_tests
[398/431] bitcoin: testing txvalidationcache_tests
[399/431] Running utility command for check-bitcoin-checkpoints_tests
[400/431] Running utility command for check-bitcoin-cuckoocache_tests
[401/431] Running utility command for check-bitcoin-txvalidationcache_tests
[402/431] bitcoin: testing radix_tests
[403/431] Running utility command for check-bitcoin-radix_tests
[404/431] bitcoin: testing ref_tests
[405/431] bitcoin: testing getarg_tests
[406/431] Running utility command for check-bitcoin-ref_tests
[407/431] bitcoin: testing crypto_tests
[408/431] Running utility command for check-bitcoin-getarg_tests
[409/431] Running utility command for check-bitcoin-crypto_tests
[410/431] bitcoin: testing blockcheck_tests
[411/431] Running utility command for check-bitcoin-blockcheck_tests
[412/431] bitcoin: testing monolith_opcodes_tests
[413/431] Running utility command for check-bitcoin-monolith_opcodes_tests
[414/431] bitcoin: testing validation_tests
[415/431] bitcoin: testing script_tests
[416/431] Running utility command for check-bitcoin-validation_tests
[417/431] Running utility command for check-bitcoin-script_tests
[418/431] bitcoin: testing validation_block_tests
[419/431] Running utility command for check-bitcoin-validation_block_tests
[420/431] bitcoin: testing skiplist_tests
[421/431] Running utility command for check-bitcoin-skiplist_tests
[422/431] bitcoin: testing util_tests
[423/431] Running utility command for check-bitcoin-util_tests
[424/431] bitcoin: testing op_reversebytes_tests
[425/431] Running utility command for check-bitcoin-op_reversebytes_tests
[426/431] bitcoin: testing transaction_tests
[427/431] Running utility command for check-bitcoin-transaction_tests
[428/431] bitcoin: testing coins_tests
[429/431] Running utility command for check-bitcoin-coins_tests
ninja: build stopped: cannot make progress due to previous errors.
Build build-clang failed with exit code 1

Unrelated coinselector failure

This revision is now accepted and ready to land.Dec 21 2020, 17:13