actions-setup-cmake icon indicating copy to clipboard operation
actions-setup-cmake copied to clipboard

Use cache to avoid API rate limit exceeded for installation ID*

Open flagarde opened this issue 2 years ago • 4 comments

Hello,

Thanks for this nice action it is very useful. However, sometimes I'm using github action to test on hardware I don't have at disposal, so this involve a lot of push, this trigger the API rate limit exceeded for installation ID* even with GITHUB_TOKEN.

Do you think it could be a solution to add an option to use 'actions/cache@v3' to avoid this situation ? like :

- name: Setup CMake
  uses: jwlawson/[email protected]
  with:
    cmake-version: '3.24.x'
    github-api-token: ${{ secrets.GITHUB_TOKEN }}
    cache : true

to download it and store it in a dedicated folder to avoid next download in case the 3.24.x is the same ?

flagarde avatar Nov 27 '22 14:11 flagarde

This is a great idea. It looks like other actions like setup-node do something like this too. I'm not sure when I'll have time to look at it, but it would be nice to have.

jwlawson avatar Dec 07 '22 20:12 jwlawson

We can confirm this issue - we often get hit by the rate limit error as well, as we need to build on many platform/compiler combinations and somewhere near the 3rd push, the rate limit error happens. An example build is here (see first attempt): https://github.com/auto-differentiation/XAD/actions/runs/5281376061/jobs/9554856112

I'm dealing with too. It happens so quickly that in practice I just can't use this action in spite of me really wanting to.

ChrisThrasher avatar Aug 17 '23 20:08 ChrisThrasher

Any news on this? I am also using this action but run into the rate limit all the time.

wjakob avatar Oct 18 '23 19:10 wjakob