mirror of
https://github.com/bitcoin/bitcoin.git
synced 2025-01-09 11:27:28 -03:00
e9b95665ee
Some checks are pending
31cc5006c3
init: Return fatal failure on snapshot validation failure (Martin Zumsande)8f1246e833
init: Improve chainstate init db error messages (TheCharlatan)cd093049dd
init: Remove incorrect comment about shutdown condition (MarcoFalke)635e9f85d7
init: Remove misleading log line when user chooses not to retry (TheCharlatan)720ce880a3
init: Improve comment describing chainstate load retry behaviour (Martin Zumsande)baea842ff1
init: Remove unneeded argument for mempool_opts checks (stickies-v) Pull request description: These are mostly followups from #30968, making the code, log lines, error messages, and comments more consistent. The last commit is an attempt at improving the error reporting when loading the chainstate. It aims to more cleanly distinguish between errors arising from a specific database, and errors where the culprit may be less clear. ACKs for top commit: achow101: ACK31cc5006c3
mzumsande: Code Review / lightly tested ACK31cc5006c3
BrandonOdiwuor: Code Review ACK31cc5006c3
. stickies-v: ACK31cc5006c3
Tree-SHA512: 59fba4845ee45a3d91bf55807ae6b1c81458463b96bf664c8b1badfac503f6b01efd52a915fc399294e68a3f69985362a5a10a3844fa23f7707145ebe9ad349b
180 lines
7.1 KiB
Python
Executable file
180 lines
7.1 KiB
Python
Executable file
#!/usr/bin/env python3
|
|
# Copyright (c) 2021-present The Bitcoin Core developers
|
|
# Distributed under the MIT software license, see the accompanying
|
|
# file COPYING or http://www.opensource.org/licenses/mit-license.php.
|
|
"""Tests related to node initialization."""
|
|
from pathlib import Path
|
|
import platform
|
|
import shutil
|
|
|
|
from test_framework.test_framework import BitcoinTestFramework, SkipTest
|
|
from test_framework.test_node import (
|
|
BITCOIN_PID_FILENAME_DEFAULT,
|
|
ErrorMatch,
|
|
)
|
|
from test_framework.util import assert_equal
|
|
|
|
|
|
class InitTest(BitcoinTestFramework):
|
|
"""
|
|
Ensure that initialization can be interrupted at a number of points and not impair
|
|
subsequent starts.
|
|
"""
|
|
|
|
def add_options(self, parser):
|
|
self.add_wallet_options(parser)
|
|
|
|
def set_test_params(self):
|
|
self.setup_clean_chain = False
|
|
self.num_nodes = 1
|
|
|
|
def init_stress_test(self):
|
|
"""
|
|
- test terminating initialization after seeing a certain log line.
|
|
- test removing certain essential files to test startup error paths.
|
|
"""
|
|
# TODO: skip Windows for now since it isn't clear how to SIGTERM.
|
|
#
|
|
# Windows doesn't support `process.terminate()`.
|
|
# and other approaches (like below) don't work:
|
|
#
|
|
# os.kill(node.process.pid, signal.CTRL_C_EVENT)
|
|
if platform.system() == 'Windows':
|
|
raise SkipTest("can't SIGTERM on Windows")
|
|
|
|
self.stop_node(0)
|
|
node = self.nodes[0]
|
|
|
|
def sigterm_node():
|
|
node.process.terminate()
|
|
node.process.wait()
|
|
|
|
def start_expecting_error(err_fragment):
|
|
node.assert_start_raises_init_error(
|
|
extra_args=['-txindex=1', '-blockfilterindex=1', '-coinstatsindex=1', '-checkblocks=200', '-checklevel=4'],
|
|
expected_msg=err_fragment,
|
|
match=ErrorMatch.PARTIAL_REGEX,
|
|
)
|
|
|
|
def check_clean_start():
|
|
"""Ensure that node restarts successfully after various interrupts."""
|
|
node.start()
|
|
node.wait_for_rpc_connection()
|
|
assert_equal(200, node.getblockcount())
|
|
|
|
lines_to_terminate_after = [
|
|
b'Validating signatures for all blocks',
|
|
b'scheduler thread start',
|
|
b'Starting HTTP server',
|
|
b'Loading P2P addresses',
|
|
b'Loading banlist',
|
|
b'Loading block index',
|
|
b'Checking all blk files are present',
|
|
b'Loaded best chain:',
|
|
b'init message: Verifying blocks',
|
|
b'init message: Starting network threads',
|
|
b'net thread start',
|
|
b'addcon thread start',
|
|
b'initload thread start',
|
|
b'txindex thread start',
|
|
b'block filter index thread start',
|
|
b'coinstatsindex thread start',
|
|
b'msghand thread start',
|
|
b'net thread start',
|
|
b'addcon thread start',
|
|
]
|
|
if self.is_wallet_compiled():
|
|
lines_to_terminate_after.append(b'Verifying wallet')
|
|
|
|
for terminate_line in lines_to_terminate_after:
|
|
self.log.info(f"Starting node and will exit after line {terminate_line}")
|
|
with node.busy_wait_for_debug_log([terminate_line]):
|
|
node.start(extra_args=['-txindex=1', '-blockfilterindex=1', '-coinstatsindex=1'])
|
|
self.log.debug("Terminating node after terminate line was found")
|
|
sigterm_node()
|
|
|
|
check_clean_start()
|
|
self.stop_node(0)
|
|
|
|
self.log.info("Test startup errors after removing certain essential files")
|
|
|
|
files_to_delete = {
|
|
'blocks/index/*.ldb': 'Error opening block database.',
|
|
'chainstate/*.ldb': 'Error opening coins database.',
|
|
'blocks/blk*.dat': 'Error loading block database.',
|
|
}
|
|
|
|
files_to_perturb = {
|
|
'blocks/index/*.ldb': 'Error loading block database.',
|
|
'chainstate/*.ldb': 'Error opening coins database.',
|
|
'blocks/blk*.dat': 'Corrupted block database detected.',
|
|
}
|
|
|
|
for file_patt, err_fragment in files_to_delete.items():
|
|
target_files = list(node.chain_path.glob(file_patt))
|
|
|
|
for target_file in target_files:
|
|
self.log.info(f"Deleting file to ensure failure {target_file}")
|
|
bak_path = str(target_file) + ".bak"
|
|
target_file.rename(bak_path)
|
|
|
|
start_expecting_error(err_fragment)
|
|
|
|
for target_file in target_files:
|
|
bak_path = str(target_file) + ".bak"
|
|
self.log.debug(f"Restoring file from {bak_path} and restarting")
|
|
Path(bak_path).rename(target_file)
|
|
|
|
check_clean_start()
|
|
self.stop_node(0)
|
|
|
|
self.log.info("Test startup errors after perturbing certain essential files")
|
|
for file_patt, err_fragment in files_to_perturb.items():
|
|
shutil.copytree(node.chain_path / "blocks", node.chain_path / "blocks_bak")
|
|
shutil.copytree(node.chain_path / "chainstate", node.chain_path / "chainstate_bak")
|
|
target_files = list(node.chain_path.glob(file_patt))
|
|
|
|
for target_file in target_files:
|
|
self.log.info(f"Perturbing file to ensure failure {target_file}")
|
|
with open(target_file, "r+b") as tf:
|
|
# Since the genesis block is not checked by -checkblocks, the
|
|
# perturbation window must be chosen such that a higher block
|
|
# in blk*.dat is affected.
|
|
tf.seek(150)
|
|
tf.write(b"1" * 200)
|
|
|
|
start_expecting_error(err_fragment)
|
|
|
|
shutil.rmtree(node.chain_path / "blocks")
|
|
shutil.rmtree(node.chain_path / "chainstate")
|
|
shutil.move(node.chain_path / "blocks_bak", node.chain_path / "blocks")
|
|
shutil.move(node.chain_path / "chainstate_bak", node.chain_path / "chainstate")
|
|
|
|
def init_pid_test(self):
|
|
BITCOIN_PID_FILENAME_CUSTOM = "my_fancy_bitcoin_pid_file.foobar"
|
|
|
|
self.log.info("Test specifying custom pid file via -pid command line option")
|
|
custom_pidfile_relative = BITCOIN_PID_FILENAME_CUSTOM
|
|
self.log.info(f"-> path relative to datadir ({custom_pidfile_relative})")
|
|
self.restart_node(0, [f"-pid={custom_pidfile_relative}"])
|
|
datadir = self.nodes[0].chain_path
|
|
assert not (datadir / BITCOIN_PID_FILENAME_DEFAULT).exists()
|
|
assert (datadir / custom_pidfile_relative).exists()
|
|
self.stop_node(0)
|
|
assert not (datadir / custom_pidfile_relative).exists()
|
|
|
|
custom_pidfile_absolute = Path(self.options.tmpdir) / BITCOIN_PID_FILENAME_CUSTOM
|
|
self.log.info(f"-> absolute path ({custom_pidfile_absolute})")
|
|
self.restart_node(0, [f"-pid={custom_pidfile_absolute}"])
|
|
assert not (datadir / BITCOIN_PID_FILENAME_DEFAULT).exists()
|
|
assert custom_pidfile_absolute.exists()
|
|
self.stop_node(0)
|
|
assert not custom_pidfile_absolute.exists()
|
|
|
|
def run_test(self):
|
|
self.init_pid_test()
|
|
self.init_stress_test()
|
|
|
|
|
|
if __name__ == '__main__':
|
|
InitTest(__file__).main()
|