Skip to content

Conversation

@lmanners
Copy link
Contributor

refs #12453

@fanquake fanquake added the Tests label May 30, 2018
@promag
Copy link
Contributor

promag commented Jun 1, 2018

Concept ACK, if the idea is to just add some anchors. I've restarted failing job.

@lmanners
Copy link
Contributor Author

lmanners commented Jun 2, 2018

Thanks @promag!

The idea behind this change is based on @jnewbery's comment here: #12453 (comment)

@jnewbery
Copy link
Contributor

Thanks @lmanners . Looks good!

Tested ACK 2ce8186.

PR description and commit log should be updated to say that this is updating logging in p2p_sendheaders.py.

@maflcko maflcko changed the title [tests] Add logging to provide anchor points when debugging failures. [tests] Add logging to provide anchor points when debugging p2p_sendheaders Jun 13, 2018
@maflcko maflcko merged commit 2ce8186 into bitcoin:master Jun 13, 2018
maflcko pushed a commit that referenced this pull request Jun 13, 2018
…ging p2p_sendheaders

2ce8186 [tests] Add logging to provide anchor points when debugging failures. (Lowell Manners)

Pull request description:

  refs #12453

Tree-SHA512: 0ad432bd848723a5b813df4d35fcd8c81d152f042499c8340a9a2a9b7918d6e785efbf9f07b63b6c0253a949044cebdb51802971c09fb8dde0efa5169f34ef20
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jul 7, 2020
…n debugging p2p_sendheaders

2ce8186 [tests] Add logging to provide anchor points when debugging failures. (Lowell Manners)

Pull request description:

  refs bitcoin#12453

Tree-SHA512: 0ad432bd848723a5b813df4d35fcd8c81d152f042499c8340a9a2a9b7918d6e785efbf9f07b63b6c0253a949044cebdb51802971c09fb8dde0efa5169f34ef20
Signed-off-by: pasta <[email protected]>

# Conflicts:
#	test/functional/sendheaders.py
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jun 28, 2021
…n debugging p2p_sendheaders

2ce8186 [tests] Add logging to provide anchor points when debugging failures. (Lowell Manners)

Pull request description:

  refs bitcoin#12453

Tree-SHA512: 0ad432bd848723a5b813df4d35fcd8c81d152f042499c8340a9a2a9b7918d6e785efbf9f07b63b6c0253a949044cebdb51802971c09fb8dde0efa5169f34ef20
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
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.

5 participants