Intermittent CI failure: "AssertionError: Fee of 0.00011000 BTC too low!" when running wallet_basic.py #19418

issue practicalswift opened this issue on June 30, 2020
  1. practicalswift commented at 6:40 AM on June 30, 2020: contributor

    Intermittent CI failure: "AssertionError: Fee of 0.00011000 BTC too low!" when running wallet_basic.py.

    See Travis log for details.

    …
2020-06-29T22:58:08.433000Z TestFramework (INFO): Initializing test directory /home/travis/build/bitcoin/bitcoin/ci/scratch/test_runner/test_runner_₿_🏃_20200629_225700/wallet_basic_150
2020-06-29T22:58:10.175000Z TestFramework (INFO): Mining blocks...
2020-06-29T22:58:11.456000Z TestFramework (INFO): test gettxout
2020-06-29T22:58:11.751000Z TestFramework (INFO): test gettxout (second part)
2020-06-29T22:58:13.291000Z TestFramework (ERROR): Assertion failed
Traceback (most recent call last):
  File "/home/travis/build/bitcoin/bitcoin/ci/scratch/build/bitcoin-x86_64-pc-linux-gnu/test/functional/test_framework/test_framework.py", line 117, in main
    self.run_test()
  File "/home/travis/build/bitcoin/bitcoin/ci/scratch/build/bitcoin-x86_64-pc-linux-gnu/test/functional/wallet_basic.py", line 204, in run_test
    node_0_bal = self.check_fee_amount(self.nodes[0].getbalance(), Decimal('20'), fee_per_byte, self.get_vsize(self.nodes[2].gettransaction(txid)['hex']))
  File "/home/travis/build/bitcoin/bitcoin/ci/scratch/build/bitcoin-x86_64-pc-linux-gnu/test/functional/wallet_basic.py", line 44, in check_fee_amount
    assert_fee_amount(fee, tx_size, fee_per_byte * 1000)
  File "/home/travis/build/bitcoin/bitcoin/ci/scratch/build/bitcoin-x86_64-pc-linux-gnu/test/functional/test_framework/util.py", line 41, in assert_fee_amount
    raise AssertionError("Fee of %s BTC too low! (Should be %s BTC)" % (str(fee), str(target_fee)))
AssertionError: Fee of 0.00011000 BTC too low! (Should be 0.00019100 BTC)
  2. practicalswift added the label Bug on Jun 30, 2020
  4. fanquake deleted a comment on Aug 13, 2020
  5. fanquake deleted a comment on Aug 13, 2020
  6. laanwj added the label Tests on Sep 3, 2020
  7. laanwj removed the label Bug on Sep 3, 2020
  8. decryp2kanon commented at 12:44 PM on October 3, 2020: contributor

    i saw same before. but now its gone with no reason? :sweat_smile:

  9. practicalswift closed this on Jun 30, 2021
  10. hebasto commented at 3:59 PM on September 28, 2021: member

    https://api.cirrus-ci.com/v1/task/4686259315539968/logs/ci.log

    11/220 - wallet_basic.py --legacy-wallet failed, Duration: 36 s
stdout:
2021-09-28T15:37:33.475000Z TestFramework (INFO): Initializing test directory /tmp/cirrus-build/ci/scratch/test_runner/test_runner_₿_🏃_20210928_153456/wallet_basic_207
2021-09-28T15:37:34.708000Z TestFramework (INFO): Mining blocks...
2021-09-28T15:37:36.232000Z TestFramework (INFO): Test gettxout
2021-09-28T15:37:36.275000Z TestFramework (INFO): Test gettxout (second part)
2021-09-28T15:37:42.855000Z TestFramework (INFO): Test sendmany
2021-09-28T15:37:44.992000Z TestFramework (INFO): Test sendmany with fee_rate param (explicit fee rate in sat/vB)
2021-09-28T15:37:45.108000Z TestFramework (INFO): Test sendmany raises 'fee rate too low' if fee_rate of 0.99999999 is passed
2021-09-28T15:37:45.115000Z TestFramework (INFO): Test sendmany raises if an invalid fee_rate is passed
2021-09-28T15:37:45.158000Z TestFramework (INFO): Test sendmany raises if an invalid conf_target or estimate_mode is passed
2021-09-28T15:37:51.913000Z TestFramework (INFO): Test -walletbroadcast
2021-09-28T15:38:02.692000Z TestFramework (INFO): Test sendtoaddress with fee_rate param (explicit fee rate in sat/vB)
2021-09-28T15:38:08.916000Z TestFramework (ERROR): Assertion failed
Traceback (most recent call last):
  File "/tmp/cirrus-build/ci/scratch/build/bitcoin-x86_64-pc-linux-gnu/test/functional/test_framework/test_framework.py", line 131, in main
    self.run_test()
  File "/tmp/cirrus-build/ci/scratch/build/bitcoin-x86_64-pc-linux-gnu/test/functional/wallet_basic.py", line 500, in run_test
    assert_fee_amount(fee, tx_size, Decimal(fee_rate_btc_kvb))
  File "/tmp/cirrus-build/ci/scratch/build/bitcoin-x86_64-pc-linux-gnu/test/functional/test_framework/util.py", line 41, in assert_fee_amount
    raise AssertionError("Fee of %s BTC too low! (Should be %s BTC)" % (str(fee), str(target_fee)))
AssertionError: Fee of 0.00000255 BTC too low! (Should be 0.00000256 BTC)
  11. hebasto reopened this on Sep 28, 2021
  12. MarcoFalke commented at 4:08 PM on September 28, 2021: member

    I was wondering if #17526 (comment) caused this, but if this has been happening for at least a year, then I am unsure.

  13. jonatack commented at 4:13 PM on September 28, 2021: member

    This test looks familiar, may have a look.

    Edit, yes it's code from 6fa72ceb802.

  14. jonatack commented at 7:43 AM on September 30, 2021: member

    api.cirrus-ci.com/v1/task/4686259315539968/logs/ci.log

    I was able to reproduce https://api.cirrus-ci.com/v1/task/4686259315539968/logs/ci.log locally yesterday after ~200 runs. I then ran wallet_basic.py overnight with #23136 and no further failures.

  15. MarcoFalke referenced this in commit 35a31d5f7e on Oct 1, 2021
  16. MarcoFalke closed this on Oct 1, 2021
  17. DrahtBot locked this on Oct 30, 2022
Contributors
practicalswifthebastodecryp2kanonMarcoFalkejonatack
Labels
Tests
Linked (view graph)
#23136 test: update fee rate assertion helper in the functional test framework
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: 2026-04-16 15:14 UTC
This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me