diff options
author | Russell Yanofsky <russ@chaincode.com> | 2016-10-24 15:33:14 -0400 |
---|---|---|
committer | MarcoFalke <falke.marco@gmail.com> | 2016-11-19 23:53:01 +0100 |
commit | 2ba5d784276783716bbf27b458514c4c3f44f4b6 (patch) | |
tree | fbb643de2edc8b9b23c7111f230958d308c261bd /qa/rpc-tests/p2p-mempool.py | |
parent | eca9b4653788570c25e646dcdfa9ba088e89f20e (diff) |
[qa] Fix bug in compactblocks v2 merge
Bug caused the wait_for_block_announcement to be called on the wrong node,
leading to nondeterminism and occasional test failures. Bug was introduced in
merge commit:
d075479 Merge #8882: [qa] Fix race conditions in p2p-compactblocks.py and sendheaders.py
Underlying commits which conflicted were:
27acfc1 [qa] Update p2p-compactblocks.py for compactblocks v2
6976db2 [qa] Another attempt to fix race condition in p2p-compactblocks.py
The first commit changed the test_compactblock_construction function signature
and second commit added code which wasn't updated during the merge to use the
new arguments.
Suhas Daftuar <sdaftuar@chaincode.com> noticed the bug and suggested the fix.
Github-Pull: #9058
Rebased-From: 47e9659ecfbe07077a4564591095bd5510e0f917
Diffstat (limited to 'qa/rpc-tests/p2p-mempool.py')
0 files changed, 0 insertions, 0 deletions