ci: failure in `wallet_importdescriptors.py ` #35544

fanquake commented at 3:34 PM on June 16, 2026: member

https://github.com/bitcoin/bitcoin/actions/runs/27626623050/job/81689877009?pr=35537:

2026-06-16T15:17:31.3998250Z [0;34m node0 2026-06-16T15:17:21.699894Z [http.00] [wallet/sqlite.cpp:55] [TraceSqlCallback] [walletdb:trace] [/Users/runner/work/bitcoin/bitcoin/repo_archive/ci/scratch_ ₿🧪_/test_runner/test_runner_₿_🏃_20260616_150224/wallet_importdescriptors_103/node0/regtest/wallets/abort_import_wallet/wallet.dat] SQLite Statement: INSERT or REPLACE into main values(?, ?) [0m
2026-06-16T15:17:31.3999070Z [0;34m node0 2026-06-16T15:17:21.699970Z [http.00] [wallet/wallet.h:939] [WalletLogPrintf] [abort_import_wallet] RescanFromTime: Rescanning last 329 blocks [0m
2026-06-16T15:17:31.3999900Z [0;34m node0 2026-06-16T15:17:21.699981Z [http.00] [wallet/wallet.h:939] [WalletLogPrintf] [abort_import_wallet] Rescan started from block 0f9188f13cb7b2c71f2a335e3a4fc328bf5beb436012afca590b1a11466e2206... (slow variant inspecting all blocks) [0m
2026-06-16T15:17:31.4000360Z [0;34m node0 2026-06-16T15:17:21.705100Z [http.00] [wallet/wallet.h:939] [WalletLogPrintf] [abort_import_wallet] Scanning current mempool transactions. [0m
2026-06-16T15:17:31.4000970Z [0;34m node0 2026-06-16T15:17:21.705121Z [http.00] [wallet/wallet.h:939] [WalletLogPrintf] [abort_import_wallet] Rescan completed in               5ms [0m
2026-06-16T15:17:31.4001230Z [0;36m test  2026-06-16T15:17:21.705640Z TestFramework (ERROR): Unexpected exception: [0m
2026-06-16T15:17:31.4001400Z [0;36m                                   Traceback (most recent call last):[0m
2026-06-16T15:17:31.4001830Z [0;36m                                     File "/Users/runner/work/bitcoin/bitcoin/repo_archive/test/functional/test_framework/test_framework.py", line 144, in main[0m
2026-06-16T15:17:31.4001970Z [0;36m                                       self.run_test()[0m
2026-06-16T15:17:31.4002090Z [0;36m                                       ~~~~~~~~~~~~~^^[0m
2026-06-16T15:17:31.4002700Z [0;36m                                     File "/Users/runner/work/bitcoin/bitcoin/repo_archive/ci/scratch_ ₿🧪_/build-aarch64-apple-darwin24.6.0/test/functional/wallet_importdescriptors.py", line 962, in run_test[0m
2026-06-16T15:17:31.4002880Z [0;36m                                       self.test_rescan_fails_import()[0m
2026-06-16T15:17:31.4003060Z [0;36m                                       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^[0m
2026-06-16T15:17:31.4003710Z [0;36m                                     File "/Users/runner/work/bitcoin/bitcoin/repo_archive/ci/scratch_ ₿🧪_/build-aarch64-apple-darwin24.6.0/test/functional/wallet_importdescriptors.py", line 193, in test_rescan_fails_import[0m
2026-06-16T15:17:31.4003990Z [0;36m                                       raise AssertionError("importdescriptors unexpectedly succeeded")[0m
2026-06-16T15:17:31.4004520Z [0;36m                                   AssertionError: importdescriptors unexpectedly succeeded[0m
2026-06-16T15:17:31.4004800Z [0;36m test  2026-06-16T15:17:21.709227Z TestFramework (DEBUG): Closing down network thread [0m

fanquake added the label CI failed on Jun 16, 2026

fanquake commented at 3:36 PM on June 16, 2026: member

cc @polespinasa

polespinasa commented at 3:43 PM on June 16, 2026: member

2026-06-16T15:17:22.3162210Z 2026-06-16T15:17:21.137717Z TestFramework (INFO): Aborting an importdescriptors rescan should fail the RPC call
2026-06-16T15:17:22.3162690Z 2026-06-16T15:17:21.705640Z TestFramework (ERROR): Unexpected exception:
2026-06-16T15:17:22.3163000Z Traceback (most recent call last):
2026-06-16T15:17:22.3163450Z   File "/Users/runner/work/bitcoin/bitcoin/repo_archive/test/functional/test_framework/test_framework.py", line 144, in main
2026-06-16T15:17:22.3163980Z     self.run_test()
2026-06-16T15:17:22.3164120Z     ~~~~~~~~~~~~~^^
2026-06-16T15:17:22.3164940Z   File "/Users/runner/work/bitcoin/bitcoin/repo_archive/ci/scratch_ ₿🧪_/build-aarch64-apple-darwin24.6.0/test/functional/wallet_importdescriptors.py", line 962, in run_test
2026-06-16T15:17:22.3165530Z     self.test_rescan_fails_import()
2026-06-16T15:17:22.3165780Z     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^
2026-06-16T15:17:22.3171280Z   File "/Users/runner/work/bitcoin/bitcoin/repo_archive/ci/scratch_ ₿🧪_/build-aarch64-apple-darwin24.6.0/test/functional/wallet_importdescriptors.py", line 193, in test_rescan_fails_import
2026-06-16T15:17:22.3173720Z     raise AssertionError("importdescriptors unexpectedly succeeded")
2026-06-16T15:17:22.3175000Z AssertionError: importdescriptors unexpectedly succeeded

The test failing is the "abort rescan" case. The test fails due to a race condition in the c++ code. It can happen that an abortrescan call is ignored by the actual rescan code.

#35512 should fix it by moving the code that makes the rescan ignore the abortrescan call

fanquake commented at 7:17 AM on June 19, 2026: member

https://github.com/bitcoin/bitcoin/actions/runs/27796442874/job/82257168562?pr=32606

Sjors commented at 3:02 PM on June 19, 2026: member

This looks like another instance: https://github.com/bitcoin/bitcoin/actions/runs/27831971948/job/82370724326?pr=32857 in #32857.