Intermittent failure in feature_config_args.py #28290

issue mzumsande opened this issue on August 17, 2023
  1. mzumsande commented at 9:09 PM on August 17, 2023: contributor

    https://cirrus-ci.com/task/5383892962639872?logs=ci#L7698

    2023-08-17T20:59:43.754000Z TestFramework (INFO): Test seed peers
    2023-08-17T21:03:45.922000Z TestFramework (ERROR): Assertion failed
    Traceback (most recent call last):
      File "/ci_container_base/ci/scratch/build/bitcoin-i686-pc-linux-gnu/test/functional/test_framework/test_framework.py", line 131, in main
        self.run_test()
      File "/ci_container_base/ci/scratch/build/bitcoin-i686-pc-linux-gnu/test/functional/feature_config_args.py", line 377, in run_test
        self.test_seed_peers()
      File "/ci_container_base/ci/scratch/build/bitcoin-i686-pc-linux-gnu/test/functional/feature_config_args.py", line 273, in test_seed_peers
        assert time.time() - start < 60
    AssertionError
    
  2. maflcko commented at 6:26 AM on August 18, 2023: member

    I also see this locally on valgrind.

    I think the general idea is that we can't measure wall clock time and assume some operation is faster than a specified wall clock duration (60 seconds in this case).

    Not sure what to do here, other than removing this check?

  3. maflcko added the label Tests on Aug 18, 2023
  4. maflcko added this to the milestone 26.0 on Aug 18, 2023
  5. mzumsande commented at 7:00 PM on September 1, 2023: contributor

    I think the general idea is that we can't measure wall clock time and assume some operation is faster than a specified wall clock duration (60 seconds in this case).

    I agree. We can't do without timeouts, but have a way to scale the timeouts for extremely slow condition (timeout_factor), so we should try to reduce instances that circumvent this. They also seem unrelated to what is being tested, so I opened #28386 to remove them.

  6. fanquake closed this on Sep 5, 2023

  7. Frank-GER referenced this in commit 1c2c80b391 on Sep 8, 2023
  8. BlackcoinDev referenced this in commit c004ba48f4 on Feb 5, 2024
  9. bitcoin locked this on Sep 4, 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: 2026-04-17 03:13 UTC

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