Differential D9662

[avalanche] Manage proof reception
ClosedPublic
Actions

Authored by Fabien on Jun 10 2021, 15:32.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Maniphest Tasks

Restricted Maniphest Task

Commits

rABCe691658b0787: [avalanche] Manage proof reception

Summary

Upon proof reception, we should create a peer (no associated node yet),
relay the proof and update the tracker status.

Depends on D9660.

Ref T1611.

Test Plan

With Clang and debug:

ninja all check-all

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.Jun 10 2021, 15:32

Herald added a reviewer: Restricted Project. · View Herald TranscriptJun 10 2021, 15:32

Fabien requested review of this revision.Jun 10 2021, 15:32

Harbormaster completed remote builds in B15967: Diff 28850.Jun 10 2021, 15:40

Build Bitcoin ABC Diffs / Diff Testing (build-without-wallet) failed.

Tail of the build log:

[0m[1;30mwallet_import_rescan.py                 | ○ Skipped | 0 s
[0m[1;30mwallet_import_with_label.py             | ○ Skipped | 0 s
[0m[1;30mwallet_importdescriptors.py             | ○ Skipped | 0 s
[0m[1;30mwallet_importmulti.py                   | ○ Skipped | 0 s
[0m[1;30mwallet_importprunedfunds.py             | ○ Skipped | 0 s
[0m[1;30mwallet_keypool.py                       | ○ Skipped | 0 s
[0m[1;30mwallet_keypool_topup.py                 | ○ Skipped | 0 s
[0m[1;30mwallet_labels.py                        | ○ Skipped | 0 s
[0m[1;30mwallet_listreceivedby.py                | ○ Skipped | 0 s
[0m[1;30mwallet_listsinceblock.py                | ○ Skipped | 0 s
[0m[1;30mwallet_listtransactions.py              | ○ Skipped | 0 s
[0m[1;30mwallet_multiwallet.py                   | ○ Skipped | 0 s
[0m[1;30mwallet_multiwallet.py --usecli          | ○ Skipped | 0 s
[0m[1;30mwallet_reorgsrestore.py                 | ○ Skipped | 0 s
[0m[1;30mwallet_resendwallettransactions.py      | ○ Skipped | 0 s
[0m[1;30mwallet_txn_clone.py                     | ○ Skipped | 0 s
[0m[1;30mwallet_txn_clone.py --mineblock         | ○ Skipped | 0 s
[0m[1;30mwallet_txn_doublespend.py               | ○ Skipped | 0 s
[0m[1;30mwallet_txn_doublespend.py --mineblock   | ○ Skipped | 0 s
[0m[1;30mwallet_watchonly.py                     | ○ Skipped | 0 s
[0m[1;30mwallet_watchonly.py --usecli            | ○ Skipped | 0 s
[0m[1;30mwallet_zapwallettxes.py                 | ○ Skipped | 0 s
[0m[1m
ALL                                     | ✓ Passed  | 376 s (accumulated) 
[0mRuntime: 86 s

----------------------------------------------------------------------
Ran 5 tests in 0.002s

OK

[160/413] Test Bitcoin RPC authentication...
...
----------------------------------------------------------------------
Ran 3 tests in 0.019s

OK
[161/413] cd /work/contrib/devtools/chainparams && /usr/bin/python3.7 ./test_make_chainparams.py
.....
----------------------------------------------------------------------
Ran 5 tests in 0.001s

OK
[169/413] Running pow test suite
PASSED: pow test suite
[181/413] Running seeder test suite
PASSED: seeder test suite
[186/413] Running bitcoin-qt test suite
PASSED: bitcoin-qt test suite
[187/413] Building CXX object src/test/CMakeFiles/test_bitcoin.dir/script_tests.cpp.o
In file included from /usr/include/boost/test/unit_test.hpp:19,
                 from ../../src/test/script_tests.cpp:30:
../../src/test/script_tests.cpp: In member function ‘void script_tests::script_build::test_method()’:
../../src/test/script_tests.cpp:540:22: note: variable tracking size limit exceeded with -fvar-tracking-assignments, retrying without
 BOOST_AUTO_TEST_CASE(script_build) {
                      ^~~~~~~~~~~~
[410/413] Running bitcoin test suite
PASSED: bitcoin test suite
ninja: build stopped: cannot make progress due to previous errors.
Build build-without-wallet failed with exit code 1

Unrelated failure

PiRK accepted this revision.Jun 11 2021, 10:27

This revision is now accepted and ready to land.Jun 11 2021, 10:27

Closed by commit rABCe691658b0787: [avalanche] Manage proof reception (authored by Fabien). · Explain WhyJun 11 2021, 19:01

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABCe691658b0787: [avalanche] Manage proof reception.