-
Notifications
You must be signed in to change notification settings - Fork 790
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Enable debug logging for core tests #4807
base: develop
Are you sure you want to change the base?
Enable debug logging for core tests #4807
Conversation
Enable NANO_LOG_STATS and debug level logging for core tests to help diagnose intermittent failures on CI runners
Test Results for Commit 71ed4ddPull Request 4807: Results Test Case Results
Last updated: 2024-12-09 09:31:06 UTC |
I wouldn't enable this, the amount of logs generated is just enormous to the point where it's not possible to easily see which tests failed anymore because github truncates viewable log size. An approach similar to gtest-parallel might be better where only log output from failed tests is printed. |
Oh right. when just running |
Gtest-parallel was added as a submodule in |
For some reason it no longer is able to show the sanitizer report https://github.com/nanocurrency/nano-node/actions/runs/12215820961/job/34078250929?pr=4807 It detects the problem, but the report is empty. Not sure what's going on here. |
c4feb1a
to
948662a
Compare
…ss executable path instead of target
948662a
to
71ed4dd
Compare
Failures on GitHub runners are sometimes hard to reproduce locally. Adding detailed logging helps analyze test failures directly from CI logs.