Skip to content

Conversation

mzumsande
Copy link
Contributor

@mzumsande mzumsande commented Oct 27, 2022

Fixes an intermittent failure in rpc_getblockfrompeer.py observed in https://cirrus-ci.com/task/6610115527704576 by adding a sync to make sure the node has processed the header we sent it before we query it for the corresponding block.

Fixes #26412

Copy link
Member

@maflcko maflcko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could use send_and_ping?

@fanquake fanquake linked an issue Oct 28, 2022 that may be closed by this pull request
@fanquake fanquake requested a review from Sjors October 28, 2022 11:02
by adding a sync to make sure the node has received the
header before we query it for the block
@mzumsande mzumsande force-pushed the 202210_testfix_blockfrompeer branch from 9ea1553 to 8a9f1e4 Compare October 28, 2022 14:19
@mzumsande
Copy link
Contributor Author

Could use send_and_ping?

Yes, that saves a line, done.

Copy link
Member

@jonatack jonatack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 8a9f1e4

@maflcko maflcko merged commit 4f270d2 into bitcoin:master Oct 29, 2022
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Oct 30, 2022
…ompeer.py

8a9f1e4 test: fix intermittent failure in rpc_getblockfrompeer.py (Martin Zumsande)

Pull request description:

  Fixes an intermittent failure in `rpc_getblockfrompeer.py` observed in https://cirrus-ci.com/task/6610115527704576 by adding a sync to make sure the node has processed the header we sent it before we query it for the corresponding block.

  Fixes bitcoin#26412

ACKs for top commit:
  jonatack:
    ACK 8a9f1e4

Tree-SHA512: f6188ab3cfd863034e44e9806d0d99a8781462bec94141501aefc71589153481ffb144e126326ab81807c2b2c93de7f4aac5d09dbcf26c4512a176e06fc6e5f8
@mzumsande mzumsande deleted the 202210_testfix_blockfrompeer branch November 3, 2022 01:10
@bitcoin bitcoin locked and limited conversation to collaborators Nov 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Failure in rpc_getblockfrompeer.py
4 participants