Page MenuHomePhabricator

refactor: use std::string for thread names
ClosedPublic

Authored by PiRK on Wed, Feb 26, 15:45.

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

PiRK requested review of this revision.Wed, Feb 26, 15:45
Fabien requested changes to this revision.Wed, Feb 26, 20:13
Fabien added a subscriber: Fabien.
Fabien added inline comments.
src/util/thread.cpp
13 ↗(On Diff #52781)

Why is that needed ?

This revision now requires changes to proceed.Wed, Feb 26, 20:13

Failed tests logs:

====== Bitcoin ABC functional tests: feature_assumevalid.py ======

------- Stdout: -------
2025-02-27T07:30:18.742000Z TestFramework (INFO): Initializing test directory /work/abc-ci-builds/build-debug/test/tmp/test_runner_₿₵_🏃_20250227_072748/feature_assumevalid_43
2025-02-27T07:31:25.242000Z TestFramework.utils (ERROR): wait_until() failed. Predicate: ''''
        self.wait_until(lambda: self.nodes[0].getblockcount() >= COINBASE_MATURITY + 1)
'''
2025-02-27T07:31:25.242000Z TestFramework (ERROR): Assertion failed
Traceback (most recent call last):
  File "/work/test/functional/test_framework/test_framework.py", line 149, in main
    self._run_test_internal()
  File "/work/test/functional/test_framework/test_framework.py", line 139, in _run_test_internal
    self.run_test()
  File "/work/test/functional/feature_assumevalid.py", line 158, in run_test
    self.wait_until(lambda: self.nodes[0].getblockcount() >= COINBASE_MATURITY + 1)
  File "/work/test/functional/test_framework/test_framework.py", line 874, in wait_until
    return wait_until_helper(
           ^^^^^^^^^^^^^^^^^^
  File "/work/test/functional/test_framework/util.py", line 296, in wait_until_helper
    raise AssertionError(
AssertionError: Predicate ''''
        self.wait_until(lambda: self.nodes[0].getblockcount() >= COINBASE_MATURITY + 1)
''' not true after 60.0 seconds
2025-02-27T07:31:25.293000Z TestFramework (INFO): Stopping nodes
2025-02-27T07:31:25.700000Z TestFramework (WARNING): Not cleaning up dir /work/abc-ci-builds/build-debug/test/tmp/test_runner_₿₵_🏃_20250227_072748/feature_assumevalid_43
2025-02-27T07:31:25.700000Z TestFramework (ERROR): Test failed. Test logging available at /work/abc-ci-builds/build-debug/test/tmp/test_runner_₿₵_🏃_20250227_072748/feature_assumevalid_43/test_framework.log
2025-02-27T07:31:25.700000Z TestFramework (ERROR): 
2025-02-27T07:31:25.700000Z TestFramework (ERROR): Hint: Call /work/test/functional/combine_logs.py '/work/abc-ci-builds/build-debug/test/tmp/test_runner_₿₵_🏃_20250227_072748/feature_assumevalid_43' to consolidate all logs
2025-02-27T07:31:25.700000Z TestFramework (ERROR): 
2025-02-27T07:31:25.700000Z TestFramework (ERROR): If this failure happened unexpectedly or intermittently, please file a bug and provide a link or upload of the combined log.
2025-02-27T07:31:25.700000Z TestFramework (ERROR): https://github.com/Bitcoin-ABC/bitcoin-abc/issues
2025-02-27T07:31:25.700000Z TestFramework (ERROR):

Each failure log is accessible here:
Bitcoin ABC functional tests: feature_assumevalid.py

This revision is now accepted and ready to land.Thu, Feb 27, 08:15
This revision was landed with ongoing or failed builds.Thu, Feb 27, 10:21
This revision was automatically updated to reflect the committed changes.