Skip to content

ci: Install specific protoc version when generating protobufs#3112

Merged
MarcoPolo merged 4 commits intomasterfrom
marco/install-protoc
Dec 18, 2024
Merged

ci: Install specific protoc version when generating protobufs#3112
MarcoPolo merged 4 commits intomasterfrom
marco/install-protoc

Conversation

@MarcoPolo
Copy link
Copy Markdown
Collaborator

@MarcoPolo MarcoPolo commented Dec 18, 2024

This fixes the issue where CI is using a different version of protoc/protoc-gen-go from our local environment. It downloads the specified version of the protobuf binary for the current system and verifies its hash. Current supported systems are macOS-aarch64 and linux-{x86_64,aarch64}. This is only used when we are regenerating the protobufs, so likely not something we will often run into.

Now the generated pbs should be reproducible locally and on CI.

@MarcoPolo
Copy link
Copy Markdown
Collaborator Author

merging without another review since this is causing every other PR to fail the check.

@MarcoPolo MarcoPolo merged commit b07e3aa into master Dec 18, 2024
2color added a commit that referenced this pull request Jan 24, 2025
…bug-nat-port-mapping

* origin/marco/bring-go-nat-home: (75 commits)
  go mod tidy
  reference internal package for mockgen
  mod tidy test-plans
  Make changes to internal nat library
  ci: move to actions/upload-artifact@v4 (#3152)
  tcpreuse: fix rcmgr accounting when tcp metrics are enabled (#3142)
  fix(net/nat): data race problem of `extAddr` (#3140)
  test: fix failing test (#3141)
  quicreuse: make it possible to use an application-constructed quic.Transport (#3122)
  nat: ignore mapping if external port is 0 (#3094)
  tcpreuse: error on using tcpreuse with pnet (#3129)
  chore: Update contribution guidelines (#3134)
  tcp: fix metrics test build directive (#3052)
  webrtc: upgrade pion/webrtc to v4 (#3098)
  ci: get back on the main release track of release checker (#3117)
  webtransport: fix docstring comment for getCurrentBucketStartTime
  chore: release v0.38.1 (#3114)
  fix(httpauth): Correctly handle concurrent requests on server (#3111)
  ci: Install specific protoc version when generating protobufs (#3112)
  fix(autorelay): Move relayFinder peer disconnect cleanup to separate goroutine (#3105)
  ...
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