Skip to content

Conversation

@snowmead
Copy link
Contributor

@snowmead snowmead commented Jan 8, 2026

Replace the arduino/setup-protoc GitHub Action with direct curl download from GitHub releases to avoid intermittent API timeout errors.

The action queries GitHub's API to list releases, which can timeout due to rate limiting or API instability on shared runners. Direct download from a known release URL bypasses the API entirely.

Using protoc v33.2 since we were already using latest version.

Example error: https://github.com/Moonsong-Labs/storage-hub/actions/runs/20828894844/job/59838287633?pr=642#step:7:20

Replace the arduino/setup-protoc GitHub Action with direct curl download
from GitHub releases to avoid intermittent API timeout errors.

The action queries GitHub's API to list releases, which can timeout due to
rate limiting or API instability on shared runners. Direct download from
a known release URL bypasses the API entirely.

Co-Authored-By: Claude Opus 4.5 <[email protected]>
@snowmead snowmead force-pushed the fix/protoc-direct-download branch from 2a74d33 to f18730a Compare January 8, 2026 19:47
@snowmead snowmead added B0-silent Changes should not be mentioned in any release notes D2-noauditneeded🙈 PR doesn't need to be audited not-breaking Does not need to be mentioned in breaking changes labels Jan 8, 2026
@snowmead snowmead marked this pull request as ready for review January 8, 2026 20:00
snowmead and others added 3 commits January 8, 2026 15:11
Replace arduino/setup-protoc@v3 action with direct curl download
of protoc v33.2 across all CI workflows to use the latest release.
@ffarall
Copy link
Collaborator

ffarall commented Jan 14, 2026

Does this mean that we'll have to manually remember to update protoc versions?

@snowmead
Copy link
Contributor Author

Does this mean that we'll have to manually remember to update protoc versions?

Yes exactly. It might not be worth the change because the error is transient, but it is frustrating when we can't run our CIs for something so menial.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

B0-silent Changes should not be mentioned in any release notes D2-noauditneeded🙈 PR doesn't need to be audited not-breaking Does not need to be mentioned in breaking changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants