qa: Failure in wallet_basic.py spendzeroconfchange test #32456

issue hodlinator openend this issue on May 9, 2025
  1. hodlinator commented at 7:48 am on May 9, 2025: contributor

    CI failure (job: “macOS 14 native, arm64, no depends, sqlite only, gui”) upon merge of my PR (ad5cd129f3cc72f2d2b140e182781bf3bb5dbacc): https://github.com/bitcoin/bitcoin/actions/runs/14916001314/job/41901630172

    Relevant high-level log:

     022/267 - wallet_basic.py failed, Duration: 23 s
     1
     2stdout:
     32025-05-08T21:04:47.763000Z TestFramework (INFO): PRNG seed is: 1820560705145373934
     42025-05-08T21:04:47.764000Z TestFramework (INFO): Initializing test directory /Users/runner/work/bitcoin/bitcoin/ci/scratch/test_runner/test_runner_₿_🏃_20250508_210317/wallet_basic_241
     52025-05-08T21:04:49.006000Z TestFramework (INFO): Mining blocks...
     62025-05-08T21:04:50.323000Z TestFramework (INFO): Test gettxout
     72025-05-08T21:04:50.342000Z TestFramework (INFO): Test gettxout (second part)
     82025-05-08T21:04:56.786000Z TestFramework (INFO): Test sendmany
     92025-05-08T21:04:56.833000Z TestFramework (INFO): Test sendmany with fee_rate param (explicit fee rate in sat/vB)
    102025-05-08T21:04:56.873000Z TestFramework (INFO): Test sendmany raises 'fee rate too low' if fee_rate of 0.99999999 is passed
    112025-05-08T21:04:56.876000Z TestFramework (INFO): Test sendmany raises if an invalid fee_rate is passed
    122025-05-08T21:04:56.894000Z TestFramework (INFO): Test sendmany raises if an invalid conf_target or estimate_mode is passed
    132025-05-08T21:04:59.702000Z TestFramework (INFO): Test -walletbroadcast
    142025-05-08T21:05:06.265000Z TestFramework (INFO): Test -reindex
    152025-05-08T21:05:08.464000Z TestFramework (INFO): Testing gettransaction response with different arguments...
    162025-05-08T21:05:08.468000Z TestFramework (INFO): Test send* RPCs with verbose=True
    172025-05-08T21:05:08.475000Z TestFramework (INFO): Test send* RPCs with verbose=False
    182025-05-08T21:05:08.480000Z TestFramework (INFO): Testing 'listunspent' outputs the parent descriptor(s) of coins
    192025-05-08T21:05:08.499000Z TestFramework (INFO): Test -spendzeroconfchange
    202025-05-08T21:05:09.800000Z TestFramework (ERROR): Assertion failed
    21Traceback (most recent call last):
    22  File "/Users/runner/work/bitcoin/bitcoin/test/functional/test_framework/test_framework.py", line 183, in main
    23    self.run_test()
    24    ~~~~~~~~~~~~~^^
    25  File "/Users/runner/work/bitcoin/bitcoin/ci/scratch/build-aarch64-apple-darwin23.6.0/test/functional/wallet_basic.py", line 659, in run_test
    26    assert_equal(len(utxos), 1)
    27    ~~~~~~~~~~~~^^^^^^^^^^^^^^^
    28  File "/Users/runner/work/bitcoin/bitcoin/test/functional/test_framework/util.py", line 77, in assert_equal
    29    raise AssertionError("not(%s)" % " == ".join(str(arg) for arg in (thing1, thing2) + args))
    30AssertionError: not(0 == 1)
    

    So far unable to reproduce on NixOS. Among other things tested:

    0build/test/functional/wallet_basic.py --random=1820560705145373934
    1build/test/functional/test_runner.py --ci -j25 --tmpdirprefix /tmp/test_runner/ --ansi --combinedlogslen=99999999 --timeout-factor=40 --quiet --failfast
    

    Edit: Next push to master succeeded as l0rinc noted, so this seems possibly timing dependent/racy.

  2. l0rinc commented at 8:50 am on May 9, 2025: contributor

    I can’t reproduce it either on my Mac - but it does seem to be passing with latest push at least.

  3. mzumsande commented at 5:03 pm on May 13, 2025: contributor
    See #32483 for a fix / way to reproduce.
  4. fanquake closed this on May 14, 2025

  5. fanquake referenced this in commit 33dfbbdff6 on May 14, 2025
  6. hodlinator commented at 1:42 pm on May 14, 2025: contributor
    @mzumsande thanks for fixing this! Sorry I didn’t have the time to fully understand how the wallet syncs before the fix was merged.

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: 2025-05-26 00:12 UTC

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