Skip to content

Conversation

darosior
Copy link
Member

Backport #32069 to 29. This is a test flakiness fix for #31757, which was backported to 29 in #32589.

Wait until the node's process has fully stopped before starting a new instance.

Since the same code is used in tool_wallet.py, this consolidates the behavior
into a 'kill_process()' function.

Github-Pull: bitcoin#32069
Rebased-From: 36b0713
@DrahtBot
Copy link
Contributor

DrahtBot commented Jul 24, 2025

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Code Coverage & Benchmarks

For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/33052.

Reviews

See the guideline for information on the review process.

Type Reviewers
ACK marcofleon

If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

@fanquake
Copy link
Member

fanquake commented Jul 24, 2025

Thanks, this should have been backported when #31757 was done, but likely missed as that happened later. Can you also add the rel note: fanquake@da24253.

@darosior
Copy link
Member Author

Sure, done.

@fanquake fanquake requested a review from marcofleon July 24, 2025 14:09
Copy link
Contributor

@marcofleon marcofleon left a comment

Choose a reason for hiding this comment

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

ACK c6fe697

@fanquake fanquake merged commit 951b349 into bitcoin:29.x Jul 24, 2025
13 of 18 checks passed
@darosior darosior deleted the 2507_backport_32069_29 branch July 24, 2025 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants