ci: tsan on clang-9 #19008

pull MarcoFalke wants to merge 2 commits into bitcoin:master from MarcoFalke:2005-ciTsanFocal changing 4 files +9 −6

MarcoFalke commented at 5:30 PM on May 18, 2020: member

Bump the compiler runtime library that includes the sanitizers from clang-8 to clang-9 to get a more recent version. Also, bump the system packages from xenial to bionic to test packages closer to what is commonly used in production.

The second commit is needed to install the tzdata package, which is missing on some operating systems. See https://travis-ci.org/github/MarcoFalke/bitcoin-core/jobs/688455828#L1727
MarcoFalke force-pushed on May 18, 2020
MarcoFalke force-pushed on May 18, 2020
DrahtBot added the label Tests on May 18, 2020
practicalswift commented at 8:03 PM on May 18, 2020: contributor

Concept ACK
hebasto commented at 10:31 PM on May 18, 2020: member

Concept ACK.
MarcoFalke force-pushed on May 18, 2020
DrahtBot commented at 12:00 AM on May 19, 2020: contributor


The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.



Conflicts

Reviewers, this pull request conflicts with the following ones:
- #18077 (net: Add NAT-PMP port forwarding support by hebasto)
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.
MarcoFalke renamed this:
~~ci: tsan on focal~~
ci: tsan on clang-9
on May 19, 2020
MarcoFalke force-pushed on May 19, 2020
ci: tsan on clang-9 fa006caa13

ci: Set DEBIAN_FRONTEND=noninteractive

Also fix travis environment variables for this build

Previously the resulting env was:

FILE_ENV="./ci/test/00_setup_env_native_tsan.sh" TEST_RUNNER_EXTRA="--exclude feature_block" #= Not= enough= memory= on= travis= machines=

faf552117e

MarcoFalke force-pushed on May 19, 2020
MarcoFalke commented at 1:01 AM on May 19, 2020: member

Shifted goalpost for now
DrahtBot cross-referenced this on May 19, 2020 from issue net: Add NAT-PMP port forwarding support by hebasto
practicalswift commented at 4:34 AM on May 19, 2020: contributor

ACK faf552117efdd456fb17b1d9490896de3b7d67a4 -- patch looks correct and Travis is happy

Thanks for doing this!
hebasto approved
hebasto commented at 7:33 AM on May 19, 2020: member

ACK faf552117efdd456fb17b1d9490896de3b7d67a4

Maybe it is time to put bionic here: https://github.com/bitcoin/bitcoin/blob/fa006caa136f05a18428c792802523637d2234ca/.travis.yml#L30 ?
fanquake merged this on May 19, 2020
fanquake closed this on May 19, 2020
MarcoFalke deleted the branch on May 19, 2020
luke-jr referenced this in commit ed95318856 on Jun 14, 2020
luke-jr referenced this in commit c8c5e03659 on Jun 15, 2020
bitcoin locked this on Feb 15, 2022

Contributors

Labels

ci: tsan on clang-9 #19008

Conflicts