-
Notifications
You must be signed in to change notification settings - Fork 37.7k
test: Run bench sanity checks in parallel with functional tests #33142
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
base: master
Are you sure you want to change the base?
Conversation
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers. Code Coverage & BenchmarksFor details see: https://corecheck.dev/bitcoin/bitcoin/pulls/33142. ReviewsSee the guideline for information on the review process. ConflictsReviewers, this pull request conflicts with the following ones:
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. |
fa1e1bd
to
fa04ef4
Compare
🚧 At least one of the CI tasks failed. HintsTry to run the tests locally, according to the documentation. However, a CI failure may still
Leave a comment here, if you need help tracking down a confusing failure. |
fa4a481
to
fabfdba
Compare
🚧 At least one of the CI tasks failed. HintsTry to run the tests locally, according to the documentation. However, a CI failure may still
Leave a comment here, if you need help tracking down a confusing failure. |
fa829da
to
faf7f97
Compare
I get an error compiling this without
|
faf7f97
to
fa9207c
Compare
thx, fixed |
Can be reviewed with the git options --color-moved=dimmed-zebra --color-moved-ws=ignore-all-space
fa9207c
to
faddf2f
Compare
(trivial rebase) |
The ctest target
bench_sanity_check
has many issues:Both issues can lead to CI timeouts and leave CPU unused during testing.
Fix all issues by running it as part of the functional tests instead. This is similar to the rpcauth tests (#32881) and util tests [bitcoin-tx, and bitcoin-util] (#32697).