8000 Upgrade to latest gRPC and latest dskit by aknuds1 · Pull Request #11492 · grafana/mimir · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Upgrade to latest gRPC and latest dskit #11492

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
May 20, 2025
Merged

Upgrade to latest gRPC and latest dskit #11492

merged 2 commits into from
May 20, 2025

Conversation

aknuds1
Copy link
Contributor
@aknuds1 aknuds1 commented May 20, 2025

What this PR does

Upgrade to gRPC@v1.72.1 (from v1.71.1) and latest dskit main (78afbb506eb9). We have to upgrade to gRPC v1.72 since the latest dskit revision depends on it.

The latest dskit revision only takes care of upgrading to gRPC v1.72, via grafana/dskit#689. I can't see any major differences in gRPC from v1.71.1 -> v1.72.1.

Which issue(s) this PR fixes or relates to

Checklist

  • Tests updated.
  • Documentation added.
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX].
  • about-versioning.md updated with experimental features.

aknuds1 added 2 commits May 20, 2025 07:27
Signed-off-by: Arve Knudsen <arve.knudsen@gmail.com>
Signed-off-by: Arve Knudsen <arve.knudsen@gmail.com>
@aknuds1 aknuds1 requested a review from Copilot May 20, 2025 05:32
Copy link
Contributor
@Copilot 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 upgrades the project's dependencies to use the latest versions of gRPC (v1.72.1) and dskit (commit 78afbb506eb9) to ensure compatibility between dskit and gRPC.

  • Updated dskit version from v0.0.0-20250508185919-68d09ac9016e to v0.0.0-20250519165046-78afbb506eb9
  • Upgraded gRPC from v1.71.1 to v1.72.1
  • Removed the exclusion for gRPC v1.72.0, allowing adoption of the new version
Comments suppressed due to low confidence (2)

go.mod:53

  • Ensure that the upgrade to gRPC v1.72.1 is thoroughly tested across all gRPC consumers in the project to catch any compatibility issues with newer API behaviors.
google.golang.org/grpc v1.72.1

go.mod:347

  • The removal of the exclusion for gRPC v1.72.0 is appropriate, but confirm that all downstream integrations have been validated against this new version.
exclude google.golang.org/grpc v1.72.0

@aknuds1 aknuds1 added the chore label May 20, 2025
@aknuds1 aknuds1 marked this pull request as ready for review May 20, 2025 05:42
@aknuds1 aknuds1 requested review from stevesg and a team as code owners May 20, 2025 05:42
@aknuds1 aknuds1 merged commit 1a106d7 into main May 20, 2025
31 checks passed
@aknuds1 aknuds1 deleted the arve/upgrade-dskit branch May 20, 2025 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0