mirror of
https://github.com/dashpay/dash.git
synced 2024-12-27 04:52:59 +01:00
Merge bitcoin/bitcoin#21792: test: Fix intermittent issue in p2p_segwit.py
fad6269916dbf8adc14d757a18f19c74e95cf659 test: Assert that exit code indicates failure (MarcoFalke) faecb72c3ca744f1adb77bd910c643cedec3b445 test: Fix intermittent issue in p2p_segwit.py (MarcoFalke) Pull request description: Calling `start_node` might call `wait_for_rpc_connection`, which will fail. https://cirrus-ci.com/task/5669555591708672?logs=ci#L3504 ``` File "/tmp/cirrus-ci-build/ci/scratch/build/bitcoin-x86_64-pc-linux-gnu/test/functional/p2p_segwit.py", line 1974, in test_upgrade_after_activation self.start_node(2, extra_args=["-reindex", f"-segwitheight={SEGWIT_HEIGHT}"]) File "/tmp/cirrus-ci-build/ci/scratch/build/bitcoin-x86_64-pc-linux-gnu/test/functional/test_framework/test_framework.py", line 508, in start_node node.wait_for_rpc_connection() File "/tmp/cirrus-ci-build/ci/scratch/build/bitcoin-x86_64-pc-linux-gnu/test/functional/test_framework/test_node.py", line 224, in wait_for_rpc_connection raise FailedToStartError(self._node_msg( test_framework.test_node.FailedToStartError: [node 2] bitcoind exited with status 1 during initialization ACKs for top commit: jnewbery: ACK fad6269916dbf8adc14d757a18f19c74e95cf659 dhruv: ACK fad6269 Tree-SHA512: 4c5e39ce25e135717ea433258518f93f09d1c528c4538a8627d3da13bc0c0ba4b45911703c26392ff0f5e0cb7831a6c7cc53e6e29102d3da9c8cfce7cef333cc
This commit is contained in:
parent
6194e454d7
commit
b2d889380c
@ -510,6 +510,7 @@ class TestNode():
|
|||||||
self.start(extra_args, stdout=log_stdout, stderr=log_stderr, *args, **kwargs)
|
self.start(extra_args, stdout=log_stdout, stderr=log_stderr, *args, **kwargs)
|
||||||
ret = self.process.wait(timeout=self.rpc_timeout)
|
ret = self.process.wait(timeout=self.rpc_timeout)
|
||||||
self.log.debug(self._node_msg(f'dashd exited with status {ret} during initialization'))
|
self.log.debug(self._node_msg(f'dashd exited with status {ret} during initialization'))
|
||||||
|
assert ret != 0 # Exit code must indicate failure
|
||||||
self.running = False
|
self.running = False
|
||||||
self.process = None
|
self.process = None
|
||||||
# Check stderr for expected message
|
# Check stderr for expected message
|
||||||
|
Loading…
Reference in New Issue
Block a user