Skip to content

chore(deps): pin grpc to v1.67.3 and stop dependabot from re-bumping it#170

Merged
albertywu merged 1 commit into
mainfrom
wua/downgrade-grpc
May 30, 2026
Merged

chore(deps): pin grpc to v1.67.3 and stop dependabot from re-bumping it#170
albertywu merged 1 commit into
mainfrom
wua/downgrade-grpc

Conversation

@albertywu
Copy link
Copy Markdown
Contributor

@albertywu albertywu commented May 30, 2026

Summary

  • Downgrade google.golang.org/grpc from v1.79.3 back to v1.67.3 (go.mod / go.sum).
  • Add .github/dependabot.yml with an ignore rule for google.golang.org/grpc so Dependabot stops opening bump PRs for it — including security-advisory PRs, which is how it got bumped to v1.79.3 last time in chore(deps): bump google.golang.org/grpc from 1.68.1 to 1.79.3 #146.
  • open-pull-requests-limit: 0 preserves the repo's current behavior of no scheduled version-update PRs; only security updates run, and grpc is now excluded from those.

Why?

grpc is a pre-existing dependency in our uber internal bazel monorepo. When we tried bumping grpc version, it caused this transitive dependency to bump as well, which has unknown risk in our monorepo:

google.golang.org/genproto/googleapis/rpc

To be safe, pin the version of grpc here to match what is used in our internal monorepo.

See internal PR for more details: https://github.com/uber-code/go-code/pull/134384

Test plan

  • make test — all 35 unit tests pass
  • make integration-test — all 8 integration tests pass
  • make e2e-test — passes
  • make check-tidy / make check-gazelle / make lint-fmt — clean against committed state
  • CI green on this PR

🤖 Generated with Claude Code

…ndabot

Downgrade from v1.79.3 to v1.67.3. The earlier auto-bump to v1.79.3
came from a Dependabot security-update PR (#146); add .github/dependabot.yml
with an ignore rule for google.golang.org/grpc so future security-advisory
PRs for this dependency are suppressed and the pin sticks.

open-pull-requests-limit: 0 keeps the prior behavior of no scheduled
version-update PRs; only security updates run, and the ignore rule excludes
grpc from those.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings May 30, 2026 03:16
@albertywu albertywu requested review from a team, behinddwalls and sbalabanov as code owners May 30, 2026 03:16
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR pins google.golang.org/grpc back to v1.67.3 and adds Dependabot configuration intended to prevent automatic grpc re-bumps while preserving security update behavior for other dependencies.

Changes:

  • Downgrades grpc in go.mod.
  • Updates go.sum to match the downgraded grpc dependency graph.
  • Adds .github/dependabot.yml with gomod update settings and a grpc ignore rule.

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.

File Description
go.mod Pins grpc to v1.67.3.
go.sum Removes checksums for transitive modules no longer required by grpc v1.67.3 and adds v1.67.3 checksums.
.github/dependabot.yml Adds Dependabot gomod configuration and grpc ignore rule.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/dependabot.yml
@albertywu albertywu added this pull request to the merge queue May 30, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks May 30, 2026
@albertywu albertywu added this pull request to the merge queue May 30, 2026
Merged via the queue into main with commit 6901f2f May 30, 2026
13 of 14 checks passed
@behinddwalls behinddwalls deleted the wua/downgrade-grpc branch May 30, 2026 15:41
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.

3 participants