Skip to content

Conversation

@maflcko
Copy link
Member

@maflcko maflcko commented Jun 5, 2017

We need to wait for node 1 to shut down completely. Otherwise we might not be able to restart the nodes and get a error:

Error: Cannot obtain a lock on data directory /tmp/testc_zc8wco/node1/regtest. Bitcoin Core is probably already running.

@maflcko maflcko added the Tests label Jun 5, 2017
@laanwj
Copy link
Member

laanwj commented Jun 6, 2017

Thanks, I noticed this error and didn't understand why it happened (assumed it was a local problem with zombie bitcoinds). utACK fa62dad

Copy link
Contributor

@gmaxwell gmaxwell left a comment

Choose a reason for hiding this comment

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

utACK

@maflcko maflcko merged commit fa62dad into bitcoin:master Jun 6, 2017
maflcko pushed a commit that referenced this pull request Jun 6, 2017
fa62dad [qa] fundrawtx: Fix shutdown race (MarcoFalke)

Tree-SHA512: bf1a3dd053a0f950639bb78fc6ef6255cc5e0b3d27efb97d956f4bd3342839c28a91d8d0793ef7473ab0c8beda30034767031060ae1895408513aa1d9f7c9ff4
@maflcko maflcko deleted the Mf1706-qaFundrawRace branch June 6, 2017 10:36
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jul 24, 2019
fa62dad [qa] fundrawtx: Fix shutdown race (MarcoFalke)

Tree-SHA512: bf1a3dd053a0f950639bb78fc6ef6255cc5e0b3d27efb97d956f4bd3342839c28a91d8d0793ef7473ab0c8beda30034767031060ae1895408513aa1d9f7c9ff4
@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.

3 participants