Allows people to fork our repo and run the tests again
Also print more cache stats
ci: Disable valgrind functionl tests on forked repos to avoid timeouts #18683
pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:2004-ciTravisNoTimeout changing 2 files +8 −2-
MarcoFalke commented at 1:03 PM on April 17, 2020: member
-
ci: Disable valgrind functionl tests on forked repos to avoid timeouts faceeae49a
-
MarcoFalke commented at 1:04 PM on April 17, 2020: member
Build in my fork: https://travis-ci.org/github/MarcoFalke/bitcoin-core/builds/676180239
- fanquake added the label Tests on Apr 17, 2020
-
DrahtBot commented at 1:24 PM on April 17, 2020: member
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--174a7506f384e20aa4161008e828411d-->
Conflicts
Reviewers, this pull request conflicts with the following ones:
- #18434 (tests: add a test-security target and run it in CI by fanquake)
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.
-
hebasto commented at 2:43 PM on April 17, 2020: member
Looks nice:
ccache version 3.4.1 cache directory /home/travis/build/bitcoin/bitcoin/ci/scratch/.ccache primary config /home/travis/build/bitcoin/bitcoin/ci/scratch/.ccache/ccache.conf secondary config (readonly) /etc/ccache.conf stats zero time Fri Apr 17 13:40:23 2020 cache hit (direct) 499 cache hit (preprocessed) 0 cache miss 3 cache hit rate 99.40 % called for link 8 cleanups performed 0 files in cache 1063 cache size 63.6 MB max cache size 100.0 MB 4.0K /home/travis/build/bitcoin/bitcoin/depends/SDKs/ 16K /home/travis/build/bitcoin/bitcoin/depends/builders/ 31M /home/travis/build/bitcoin/bitcoin/depends/built/ 24K /home/travis/build/bitcoin/bitcoin/depends/hosts/ 108K /home/travis/build/bitcoin/bitcoin/depends/packages/ 76K /home/travis/build/bitcoin/bitcoin/depends/patches/ 4.0K /home/travis/build/bitcoin/bitcoin/depends/sdk-sources/ 4.0K /home/travis/build/bitcoin/bitcoin/depends/sources/ 255M /home/travis/build/bitcoin/bitcoin/depends/x86_64-pc-linux-gnu/ 214M /home/travis/build/bitcoin/bitcoin/releases/x86_64-pc-linux-gnu -
MarcoFalke commented at 3:49 PM on April 17, 2020: member
valgrind passed in forked repo: https://travis-ci.org/github/MarcoFalke/bitcoin-core/jobs/676180248#L3100
valgrind run in this repo: https://travis-ci.org/github/bitcoin/bitcoin/jobs/676181073#L3107
- hebasto approved
-
hebasto commented at 3:58 PM on April 17, 2020: member
ACK faceeae49ae85fb644996fedb970ad9d0e297ca5, tested on my own repo: https://travis-ci.org/github/hebasto/bitcoin/jobs/676257500
- MarcoFalke deleted a comment on Apr 17, 2020
- MarcoFalke merged this on Apr 17, 2020
- MarcoFalke closed this on Apr 17, 2020
- MarcoFalke deleted the branch on Apr 17, 2020
- sidhujag referenced this in commit a9e929106b on Apr 17, 2020
-
jonatack commented at 6:11 PM on April 17, 2020: member
Good idea.
ACK faceeae49
- DrahtBot locked this on Feb 15, 2022
Contributors
Labels