ci: Only use credits for pull requests to the main repo #20680

pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:2012-ciCredits changing 1 files +15 −10

MarcoFalke commented at 8:34 AM on December 17, 2020: member
No need to spend credits for the master branch, because the build shouldn't fail there anyway and it is not time-critical to get a fast feedback.

Some other changes:
- Disable stateful for faster scheduling
- Reduce lint memory from 8G to 1G for faster scheduling
ci: Only use credits for pull requests to the main repo facf5e37f6
DrahtBot added the label Tests on Dec 17, 2020

in .cirrus.yml:46 in facf5e37f6

  39 | @@ -41,6 +40,11 @@ global_task_template: &GLOBAL_TASK_TEMPLATE
  40 |    ci_script:
  41 |      - ./ci/test_run_all.sh
  42 |  
  43 | +compute_credits_template: &CREDITS_TEMPLATE
  44 | +  # https://cirrus-ci.org/pricing/#compute-credits
  45 | +  # Only use credits for pull requests to the main repo
  46 | +  use_compute_credits: $CIRRUS_REPO_FULL_NAME == 'bitcoin/bitcoin' && $CIRRUS_PR != ""

hebasto commented at 9:10 AM on December 17, 2020:

Are pulls into the GUI repo skipped intentionally?

MarcoFalke commented at 9:20 AM on December 17, 2020:

Yes, it is a separate org, so billing would need to be set up separately. Is there a lot of traffic on the gui repo, so that this is needed?

hebasto commented at 9:22 AM on December 17, 2020:

No much traffic :)

hebasto commented at 9:12 AM on December 17, 2020: member

Concept ACK.
hebasto approved
hebasto commented at 9:23 AM on December 17, 2020: member

ACK facf5e37f619be6bd7f8768058842679cd9ad933, I have reviewed the code and it looks OK, I agree it can be merged.
MarcoFalke merged this on Dec 17, 2020
MarcoFalke closed this on Dec 17, 2020
MarcoFalke deleted the branch on Dec 17, 2020
DrahtBot locked this on Feb 15, 2022

Contributors

MarcoFalke

hebasto

Labels

Tests