test: Remove cached directories and associated script blocks from appveyor config #19444

pull sipsorcery wants to merge 1 commits into bitcoin:master from sipsorcery:appveyor-remove-cache changing 1 files +19 −39
  1. sipsorcery commented at 2:01 pm on July 4, 2020: member

    Appveyor CI jobs have been failing in the last 24 hours due to a seemingly corrupted cache, see #19440.

    It’s possible that the appveyor cache issue is related to the recent update of the Visual Studio 2019 image

    PR #19431 changes the “save cache or error” to false in an attempt to avoid a failing CI job from potentially corrupting the cache. In theory the only way a PR could affect the cache is if the vcpkg install list changed. That happens very rarely and did not happen in the last 24 hours and so was not the cause of the current cache problems.

    I have done some testing with appveyor build jobs on my own fork and found that installing the vcpkg dependencies from scratch and doing a full build can now be done in just under 60 minutes. This is the first time in over 5 months I have been able to build Bitcoin Core on appveyor. Either the new Visual Studio 2019 image has dramatically reduced the build time or appveyor images have had their CPU increased.

    This PR removes all use of dependency caching from the appveyor CI config. The trade-off is the 15 minutes saved on each build from having the dependencies cached versus the hours maintainers need to spend investigating when the CI jobs start failing.

  2. Remove cached directories and associated script blocks from appveyor CI configuration. 961e667600
  3. MarcoFalke commented at 2:36 pm on July 4, 2020: member

    The functional tests were disabled a while ago, so the total run time is lower

    Concept ACK. I did like the 10 minute speedup, but if this causes too much issues, removing seems fine for now.

  4. sipsorcery commented at 2:41 pm on July 4, 2020: member

    The functional tests were disabled a while ago, so the total run time is lower

    Ah that explains it. I saw some test results and assumed it was still the original command.

  5. DrahtBot commented at 2:50 pm on July 4, 2020: member

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

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #19431 (ci: Avoid failing pull requests destory the appveyor cache by MarcoFalke)

    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.

  6. DrahtBot added the label Tests on Jul 4, 2020
  7. MarcoFalke commented at 6:12 pm on July 4, 2020: member

    ACK 961e667600441c35845fcb36b120500c813cd3ed

    reviewed with https://github.com/bitcoin/bitcoin/pull/19444/files?w=1

  8. MarcoFalke merged this on Jul 4, 2020
  9. MarcoFalke closed this on Jul 4, 2020

  10. hebasto commented at 5:43 am on July 5, 2020: member
    post-merge ACK 961e667600441c35845fcb36b120500c813cd3ed, works fine in my forked repo: https://ci.appveyor.com/project/hebasto/bitcoin/builds/33907119
  11. MarcoFalke referenced this in commit fae0e93836 on Jul 10, 2020
  12. fanquake referenced this in commit 8c0a9595ec on Aug 11, 2020
  13. MarcoFalke referenced this in commit 28a9df7d76 on Aug 11, 2020
  14. backpacker69 referenced this in commit 9e8865d715 on Sep 8, 2020
  15. Bushstar referenced this in commit a6c3f661f5 on Oct 21, 2020
  16. Platinumwrist referenced this in commit 05e2740d92 on Oct 25, 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-11-17 12:12 UTC

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