The message ERROR: Failed to reject proof is printed when a proof is rejected due to conflicts. This diff fixes the issue by simply using the output from the PeerManager::rejectProof() method instead of another layer of logic.
Details
Details
- Reviewers
sdulfari - Group Reviewers
Restricted Project - Commits
- rABCd5f6df1596a6: [avalanche] Fix an log error during proof rejection
./test/functional/test_runner.py abc_p2p_avalanche_proof_voting
Diff Detail
Diff Detail
- Repository
- rABC Bitcoin ABC
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Comment Actions
Build Bitcoin ABC Diffs / Diff Testing (build-without-wallet) failed.
Failed tests logs:
====== Bitcoin ABC functional tests: abc_p2p_compactproofs.py ======
------- Stdout: -------
2022-07-15T13:36:10.691000Z TestFramework (INFO): Initializing test directory /work/abc-ci-builds/build-without-wallet/test/tmp/test_runner_₿₵_ _20220715_133516/abc_p2p_compactproofs_17
2022-07-15T13:36:11.383000Z TestFramework (INFO): Check we send a getavaproofs message to our avalanche outbound peers
2022-07-15T13:36:13.374000Z TestFramework (INFO): Check we send periodic getavaproofs message to some of our peers
2022-07-15T13:36:13.982000Z TestFramework (INFO): After the first avaproofs has been received, all the peers are requested periodically
2022-07-15T13:36:18.690000Z TestFramework (ERROR): Assertion failed
Traceback (most recent call last):
File "/work/test/functional/test_framework/test_framework.py", line 136, in main
self.run_test()
File "/work/test/functional/abc_p2p_compactproofs.py", line 674, in run_test
self.test_send_outbound_getavaproofs()
File "/work/test/functional/abc_p2p_compactproofs.py", line 190, in test_send_outbound_getavaproofs
p.send_message(build_msg_avaproofs([]))
File "/usr/lib/python3.7/contextlib.py", line 119, in __exit__
next(self.gen)
File "/work/test/functional/test_framework/test_node.py", line 535, in assert_debug_log
str(expected_msgs), print_log))
File "/work/test/functional/test_framework/test_node.py", line 213, in _raise_assertion_error
raise AssertionError(self._node_msg(msg))
AssertionError: [node 0] Expected messages "['received: avaproofs']" does not partially match log:
-
2022-07-15T13:36:18.745000Z TestFramework (INFO): Stopping nodes
2022-07-15T13:36:19.499000Z TestFramework (WARNING): Not cleaning up dir /work/abc-ci-builds/build-without-wallet/test/tmp/test_runner_₿₵_ _20220715_133516/abc_p2p_compactproofs_17
2022-07-15T13:36:19.499000Z TestFramework (ERROR): Test failed. Test logging available at /work/abc-ci-builds/build-without-wallet/test/tmp/test_runner_₿₵_ _20220715_133516/abc_p2p_compactproofs_17/test_framework.log
2022-07-15T13:36:19.499000Z TestFramework (ERROR):
2022-07-15T13:36:19.499000Z TestFramework (ERROR): Hint: Call /work/test/functional/combine_logs.py '/work/abc-ci-builds/build-without-wallet/test/tmp/test_runner_₿₵_ _20220715_133516/abc_p2p_compactproofs_17' to consolidate all logs
2022-07-15T13:36:19.499000Z TestFramework (ERROR):
2022-07-15T13:36:19.499000Z TestFramework (ERROR): If this failure happened unexpectedly or intermittently, please file a bug and provide a link or upload of the combined log.
2022-07-15T13:36:19.499000Z TestFramework (ERROR): https://github.com/Bitcoin-ABC/bitcoin-abc/issues
2022-07-15T13:36:19.499000Z TestFramework (ERROR):Each failure log is accessible here:
Bitcoin ABC functional tests: abc_p2p_compactproofs.py