feature_block.py is being run in the tsan job, i.e here, even though it should be excluded. My hasty assumption is that this will fix it. In any case, all other instances of TEST_RUNNER_EXTRA seem to have moved out of .travis.yml and into the different CI configurations.
test: move TEST_RUNNER_EXTRA into native tsan setup #19412
pull fanquake wants to merge 1 commits into bitcoin:master from fanquake:move_tsan_test_runner_config changing 2 files +1 −3-
fanquake commented at 2:03 PM on June 29, 2020: member
-
test: move TEST_RUNNER_EXTRA into native tsan setup a92e48b02d
- fanquake added the label Tests on Jun 29, 2020
-
MarcoFalke commented at 2:08 PM on June 29, 2020: member
Thx. Good catch!
-
MarcoFalke commented at 2:09 PM on June 29, 2020: member
ACK a92e48b02df545e620a7b1de74f647f46413d3fb
-
MarcoFalke commented at 2:11 PM on June 29, 2020: member
Looks like the bug has been introduced in commit fa74a54fad7abfe3b0c98c5a6e4780d63d35b13f
-
practicalswift commented at 2:27 PM on June 29, 2020: contributor
ACK a92e48b02df545e620a7b1de74f647f46413d3fb -- patch looks correct
Good catch!
- hebasto approved
-
hebasto commented at 2:42 PM on June 29, 2020: member
ACK a92e48b02df545e620a7b1de74f647f46413d3fb, I have reviewed the code and it looks OK, I agree it can be merged.
- MarcoFalke closed this on Jun 29, 2020
- MarcoFalke reopened this on Jun 29, 2020
-
MarcoFalke commented at 4:47 PM on June 29, 2020: member
Looks like the first run passed, but it took more than 2 hours: https://travis-ci.org/github/bitcoin/bitcoin/jobs/703160129
Seems odd, but something to investigate later.
- MarcoFalke merged this on Jun 29, 2020
- MarcoFalke closed this on Jun 29, 2020
- fanquake deleted the branch on Jun 30, 2020
- DrahtBot locked this on Feb 15, 2022
Contributors
Labels