Skip to content

Fix workflows failing because releases weren't being found#130

Merged
crschardt merged 4 commits intoPhotonVision:mainfrom
crschardt:fix/missing-releases
Mar 2, 2026
Merged

Fix workflows failing because releases weren't being found#130
crschardt merged 4 commits intoPhotonVision:mainfrom
crschardt:fix/missing-releases

Conversation

@crschardt
Copy link
Contributor

@crschardt crschardt commented Mar 2, 2026

GitHub rate limits API requests. The install.sh script uses the API through curl to determine the download URL for the correct PhotonVision jar file for each image. Building the images resulted in enough closely spaced calls to trigger the rate limit for some runners with the resulting response from the server:

{"message":"API rate limit exceeded for XXX.XXX.XXX.XXX. (But here'\''s the good news: Authenticated requests get a higher rate limit. Check out the documentation for more details.)","documentation_url":"https://docs.github.com/rest/overview/resources-in-the-rest-api#rate-limiting"}

The rate limit is fairly low for anonymous requests and much higher for authenticated ones. This PR fixes the problem by updating the install.sh script to use the GITHUB_TOKEN to authenticate the request when running as a workflow.

@crschardt crschardt changed the title Investigate workflows failing for missing releases Fix workflows failing because releases weren't being found Mar 2, 2026
@crschardt crschardt marked this pull request as ready for review March 2, 2026 04:42
@crschardt crschardt merged commit e5018c1 into PhotonVision:main Mar 2, 2026
18 checks passed
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.

2 participants