Skip to content
Browse files

[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 bitcoin#8882: [qa] Fix race conditions in and

Underlying commits which conflicted were:

27acfc1 [qa] Update for compactblocks v2
6976db2 [qa] Another attempt to fix race condition in

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 <> noticed the bug and suggested the fix.
  • Loading branch information...
ryanofsky committed Oct 24, 2016
1 parent 273bde3 commit 47e9659ecfbe07077a4564591095bd5510e0f917
Showing with 2 additions and 2 deletions.
  1. +2 −2 qa/rpc-tests/
@@ -300,8 +300,8 @@ def test_compactblock_construction(self, node, test_node, version, use_witness_a
assert(segwit_tx_generated) # check that our test is not broken

# Wait until we've seen the block announcement for the resulting tip
tip = int(self.nodes[0].getbestblockhash(), 16)
tip = int(node.getbestblockhash(), 16)

# Now mine a block, and look at the resulting compact block.

0 comments on commit 47e9659

Please sign in to comment.
You can’t perform that action at this time.