ci: Run tsan ci config on cirrus #19424

pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:2006-ciTsanCirrus changing 2 files +14 −7
  1. MarcoFalke commented at 4:03 pm on July 1, 2020: member

    Fixes bitcoin-core/gui#12

    Copied description from #19321:

    Currently it is not possible to use travis in forked repositories due to the 50 minute limit on builds. A fresh build (uncached) of the thread sanitizer config takes more than 50 minutes.

    One approach to fix this could be to throw away tests until the run time is less than 50 minutes. However, the risk of being blind of failures in the thrown away tests is not worth the gain. Also, to detect them, one has to run the tsan configuration nightly and failures could only be detected post-merge.

    Another approach would be to ask travis support to raise the limit for a forked repository. This is a tedious and manual one-by-one process, so I’d rather not.

    Finally, a different ci provider can be used, since the config files are designed to be platform-agnostic. This is what I picked.

    I kept all settings identical to the travis machine for now. Both providers run in the google cloud, so this should be a “move-only”.

  2. DrahtBot added the label Tests on Jul 1, 2020
  3. DrahtBot commented at 5:45 pm on July 1, 2020: member

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

    Conflicts

    No conflicts as of last run.

  4. MarcoFalke force-pushed on Jul 2, 2020
  5. DrahtBot added the label Needs rebase on Jul 2, 2020
  6. ci: Run tsan ci config on cirrus fa8e6df282
  7. MarcoFalke force-pushed on Jul 2, 2020
  8. MarcoFalke commented at 5:03 pm on July 2, 2020: member
    Rebased
  9. DrahtBot removed the label Needs rebase on Jul 2, 2020
  10. fanquake approved
  11. fanquake commented at 9:45 am on July 3, 2020: member
    ACK fa8e6df282af0d396d75b03721f1b59a520ced19 - my understanding is that test coverage remains the same. Just swapping providers to work-around the Travis time-limit in other repos.
  12. MarcoFalke merged this on Jul 3, 2020
  13. MarcoFalke closed this on Jul 3, 2020

  14. MarcoFalke deleted the branch on Jul 3, 2020
  15. MarcoFalke referenced this in commit c6532fa6c1 on Aug 19, 2020
  16. sidhujag referenced this in commit d2a49e79d0 on Aug 19, 2020
  17. DrahtBot locked this on Feb 15, 2022

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-07-01 10:13 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me