travis: Avoid timeout on verify-commits check #14705

pull MarcoFalke wants to merge 2 commits into bitcoin:master from MarcoFalke:Mf1811-travisVerifyCommits changing 2 files +3 −2
  1. MarcoFalke commented at 5:14 PM on November 10, 2018: member

    The verify-commits check is too expensive to run in full (calculate Tree-SHA512 and clean-merge for every single merge commit in history) every day (the cron job runs every ~24h). Since the cron job is running every day, it is also redundant to redo most of the work on the next day.

    So, only check two days worth of commits and assume that travis checked the Tree-SHA512 and clean-merge for all other commits already. The script will still check all the signatures, since the check-result for them depends on external inputs such as current time or the public keys we got from the server.

    [Note that travis is not meant to do the verification for anyone or is meant to be trusted in any way. This check only serves as a belt-and-suspender to notify maintainers in case of a technical issue or script malfunction. But since the script is timing out for months now, its purpose is diminished right now.]

  2. MarcoFalke added the label Tests on Nov 10, 2018
  3. MarcoFalke referenced this in commit 8427721a64 on Nov 10, 2018
  4. MarcoFalke force-pushed on Nov 10, 2018
  5. travis: Avoid timeout on verify-commits check fa3df025e1
  6. MarcoFalke force-pushed on Nov 10, 2018
  7. MarcoFalke referenced this in commit 0937241f2b on Nov 10, 2018
  8. MarcoFalke force-pushed on Nov 10, 2018
  9. qa: Raise ci test_runner timeout to 40 mins fa5a6ce102
  10. MarcoFalke commented at 6:17 PM on November 10, 2018: member
  11. ken2812221 commented at 9:25 PM on November 10, 2018: contributor

    utACK fa5a6ce1022acf4722832490b1c6efb2b256e5ad

  12. sipa commented at 2:45 AM on November 11, 2018: member

    Concept ACK

  13. promag commented at 2:15 PM on November 12, 2018: member

    utACK fa5a6ce.

  14. MarcoFalke referenced this in commit e0c6bb7823 on Nov 13, 2018
  15. MarcoFalke merged this on Nov 13, 2018
  16. MarcoFalke closed this on Nov 13, 2018

  17. MarcoFalke deleted the branch on Nov 13, 2018
  18. knst referenced this in commit 494b6a3a38 on Aug 10, 2021
  19. knst referenced this in commit e28c83c43e on Aug 10, 2021
  20. knst referenced this in commit b2305dc612 on Aug 16, 2021
  21. PastaPastaPasta referenced this in commit 6699b4d79a on Aug 23, 2021
  22. UdjinM6 referenced this in commit f38aa44bfd on Aug 29, 2021
  23. DrahtBot locked this on Sep 8, 2021

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: 2026-04-17 06:15 UTC

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