Repository navigation
Replace arduino/setup-protoc with direct protoc download - #96
Merged
Merged
Conversation
arduino/setup-protoc@v3 fails with 'unable to get latest version' even with a pinned version and repo-token set (confirmed reproducing consistently across networkservicemesh/sdk and networkservicemesh/sdk-kernel CI runs, on the correct merged commit). The action's computeVersion() always calls the GitHub releases API to resolve/validate the pinned version, regardless of the token, and this call is failing in the GitHub Actions runner environment. Download the pinned protoc release archive directly from GitHub instead, bypassing that API call entirely. This is a superseding fix for #95. Signed-off-by: Botond Szirtes <botond.szirtes@est.tech>
bszirtes
added a commit
to networkservicemesh/sdk-kernel
that referenced
this pull request
Aug 18, 2026
arduino/setup-protoc@v3 fails with 'unable to get latest version' even with the version pinned and repo-token set (see networkservicemesh/.github#95 and #96). Inline the gogenerate job locally with a direct protoc download to unblock CI on this release branch. TODO: revert once networkservicemesh/.github#96 is merged and picked up by the @main ref. Signed-off-by: Botond Szirtes <botond.szirtes@est.tech>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Replaces the
arduino/setup-protoc@v3step insdk-gogenerate.yamlwith a directcurl+unzipof the pinned protoc release archive.Why
arduino/setup-protoc@v3consistently fails with:This happens even with the version pinned (
3.8.0) andrepo-tokenset (#95). The action'scomputeVersion()always calls the GitHub releases API to resolve/validate the version regardless of pinning, and that call is failing in the GitHub Actions runner environment — reproduced consistently acrossnetworkservicemesh/sdkandnetworkservicemesh/sdk-kernelCI runs, confirmed running on the commit that already includes the repo-token fix from #95.Downloading the pinned protoc release archive directly from
github.com/protocolbuffers/protobuf/releasesavoids that API call entirely, so it can't hit this failure mode. Verified the download URL is reachable and returns the correct archive:https://github.com/protocolbuffers/protobuf/releases/download/v3.8.0/protoc-3.8.0-linux-x86_64.zipThis supersedes #95 (repo-token alone wasn't sufficient).
Caller impact
None. Same as #95 — no caller repo (
sdk,sdk-kernel, etc.) needs anyci.yamlchanges.Testing
ubuntu-latest(Linux x86_64) since this job always runs onubuntu-latest.