Make ctest stops when the first failure happens.
Wasting less resources and notifying the developer faster when a failure occurs.
ci: make ctest stop on failure #31257
pull furszy wants to merge 1 commits into bitcoin:master from furszy:2024_ctest_stop_on_failure changing 2 files +3 −3-
furszy commented at 6:12 PM on November 8, 2024: member
-
36a22e5683
ci: make ctest stop on failure
Wastes less resources and notifies the developer faster when a failure occurs.
-
DrahtBot commented at 6:12 PM on November 8, 2024: contributor
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--006a51241073e994b41acfe9ec718e94-->
Code Coverage & Benchmarks
For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/31257.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline for information on the review process.
If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
<!--174a7506f384e20aa4161008e828411d-->
Conflicts
Reviewers, this pull request conflicts with the following ones:
- #31182 (ci: Place datadirs for tests under tmpfs by maflcko)
- #28710 (Remove the legacy wallet and BDB dependency by achow101)
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.
- DrahtBot added the label Tests on Nov 8, 2024
-
maflcko commented at 12:10 PM on November 9, 2024: member
lgtm ACK 36a22e5683375b7925767de9daa9df4c48831c15
I think the same happened when the project was using
make, but I haven't double-checked. - tdb3 approved
-
tdb3 commented at 7:08 PM on November 10, 2024: contributor
code review and test ACK 36a22e5683375b7925767de9daa9df4c48831c15
- Grepped for
ctestin other directories (in case other files should be adjusted). Didn't see anything else to update. - Sanity checked by inserting an intentional failure (
BOOST_CHECK_EQUAL(1, 2)) intoaddrman_simpleand ranctestlocally as defined in the CI ymls to confirm the commands with--stop-on-failurebehave as expected (i.e. stop early). Behaved as expected.
- Grepped for
- fanquake merged this on Nov 11, 2024
- fanquake closed this on Nov 11, 2024
- TheCharlatan referenced this in commit a73b2bd0f0 on Nov 14, 2024
- bug-castercv502 referenced this in commit fdcc066ca0 on Sep 28, 2025
- bitcoin locked this on Nov 11, 2025