Skip to content

ci: Authenticate arduino/setup-task API requests - #738

Merged
cdecker merged 1 commit into
mainfrom
2026w32-ci-setup-task-auth
Aug 3, 2026
Merged

ci: Authenticate arduino/setup-task API requests#738
cdecker merged 1 commit into
mainfrom
2026w32-ci-setup-task-auth

Conversation

@cdecker

@cdecker cdecker commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

arduino/setup-task queries the GitHub API to resolve and download the Task
release. Unauthenticated, those requests are rate limited per runner IP, and
that budget is shared across all GitHub-hosted runners. When it is exhausted
the step fails a few seconds into the job with:

##[error]API rate limit exceeded for 4.155.99.32. (But here's the good news:
Authenticated requests get a higher rate limit. ...)

This took down the ClnVersionManager / source job on an unrelated PR, and
will keep doing so at random. Passing repo-token authenticates the requests
so they get the much larger per-token allowance.

check-self.yml already tried to do this, but set GH_TOKEN in the step's
env. The action reads only the repo-token input — its action.yml
declares no env fallback — so that had no effect and the job was
unauthenticated too. Switched to the input.

@cdecker
cdecker force-pushed the 2026w32-ci-setup-task-auth branch from 77aa024 to 0808819 Compare August 3, 2026 10:17
The action queries the GitHub API to resolve and download the Task
release. Unauthenticated, those requests are rate limited per runner IP,
which is shared across all GitHub-hosted runners. When that budget is
exhausted the step fails with "API rate limit exceeded for <ip>" a few
seconds into the job, failing otherwise healthy pipelines.

Pass `repo-token` so the requests are authenticated and get the larger
per-token allowance.

`check-self.yml` already attempted this, but set `GH_TOKEN` in the step
env. The action only reads the `repo-token` input, so it had no effect
and that job was unauthenticated too.
@cdecker
cdecker force-pushed the 2026w32-ci-setup-task-auth branch from 0808819 to 50dd166 Compare August 3, 2026 10:49
@cdecker
cdecker merged commit 276943c into main Aug 3, 2026
16 checks passed
@cdecker
cdecker deleted the 2026w32-ci-setup-task-auth branch August 3, 2026 11:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant