ci: failure in wallet_assumeutxo.py --descriptors #29234

issue fanquake openend this issue on January 11, 2024
  1. fanquake commented at 5:32 pm on January 11, 2024: member

    Merged in #28838. Seems to be failing on all MSVC CI jobs.

    https://github.com/bitcoin/bitcoin/actions/runs/7492122146/job/20394956531?pr=29232#step:27:7862:

     0  test  2024-01-11T17:24:30.783000Z TestFramework (INFO): Backup can't be loaded during background sync 
     1 test  2024-01-11T17:24:30.783000Z TestFramework (ERROR): Assertion failed 
     2                                   Traceback (most recent call last):
     3                                     File "D:\a\bitcoin\bitcoin\test\functional\test_framework\util.py", line 140, in try_rpc
     4                                       fun(*args, **kwds)
     5                                     File "D:\a\bitcoin\bitcoin\test\functional\test_framework\coverage.py", line 50, in __call__
     6                                       return_val = self.auth_service_proxy_instance.__call__(*args, **kwargs)
     7                                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
     8                                     File "D:\a\bitcoin\bitcoin\test\functional\test_framework\authproxy.py", line 129, in __call__
     9                                       raise JSONRPCException(response['error'], status)
    10                                   test_framework.authproxy.JSONRPCException: remove_all: The process cannot access the file because it is being used by another process.: "D:\a\_temp\test_runner_?_??_20240111_171305\wallet_assumeutxo_47\node1\regtest\w" (-1)
    11                                   During handling of the above exception, another exception occurred:
    12                                   Traceback (most recent call last):
    13                                     File "D:\a\bitcoin\bitcoin\test\functional\test_framework\test_framework.py", line 131, in main
    14                                       self.run_test()
    15                                     File "D:\a\bitcoin\bitcoin/test/functional/wallet_assumeutxo.py", line 127, in run_test
    16                                       assert_raises_rpc_error(-4, "Wallet loading failed. Error loading wallet. Wallet requires blocks to be downloaded, and software does not currently support loading wallets while blocks are being downloaded out of order when using assumeutxo snapshots. Wallet should be able to load successfully after node sync reaches height 299", n1.restorewallet, "w", "backup_w.dat")
    17                                     File "D:\a\bitcoin\bitcoin\test\functional\test_framework\util.py", line 131, in assert_raises_rpc_error
    18                                       assert try_rpc(code, message, fun, *args, **kwds), "No exception raised"
    19                                              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    20                                     File "D:\a\bitcoin\bitcoin\test\functional\test_framework\util.py", line 144, in try_rpc
    21                                       raise AssertionError("Unexpected JSONRPC error code %i" % e.error["code"])
    22                                   AssertionError: Unexpected JSONRPC error code -1
    
  2. fanquake added the label Windows on Jan 11, 2024
  3. fanquake added the label Tests on Jan 11, 2024
  4. jonatack commented at 7:28 pm on January 11, 2024: contributor
  5. maflcko commented at 2:33 pm on January 12, 2024: member
    Could this be a bug in the wallet? Why would it call remove_all on the wallet file of a completely different node?
  6. achow101 commented at 4:48 pm on January 12, 2024: member

    Could this be a bug in the wallet?

    Indeed, this looks more like a windows related bug when there’s a wallet restoring failure.

    Why would it call remove_all on the wallet file of a completely different node?

    I don’t think it is. I think the problem is more along the lines of the same bitcoind has the database open at the time it tries to cleanup the failed restore.

  7. achow101 commented at 1:14 am on January 13, 2024: member
    This actually revealed quite a subtle bug in the way that we handle cleaning up after an AttachChain failure. I’ve opened a fix in #29243
  8. maflcko removed the label Windows on Jan 13, 2024
  9. maflcko removed the label Tests on Jan 13, 2024
  10. maflcko added the label Wallet on Jan 13, 2024
  11. maflcko added the label Bug on Jan 13, 2024
  12. bitcoin deleted a comment on Jan 14, 2024
  13. fanquake closed this on Jan 15, 2024


github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-09-28 22:12 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me