Skip to content

Support Go gRPC 1.77 - #955

Merged
grcevski merged 5 commits into
open-telemetry:mainfrom
grcevski:grcp_1.77
Dec 2, 2025
Merged

Support Go gRPC 1.77#955
grcevski merged 5 commits into
open-telemetry:mainfrom
grcevski:grcp_1.77

Conversation

@grcevski

@grcevski grcevski commented Dec 1, 2025

Copy link
Copy Markdown
Contributor

gRPC 1.77 for Go embedded again the Stream pointer in the ServerStream, from 1.77.0 we don't double dereference (which we needed to do for 1.69.0+).

Related Go gRPC change grpc/grpc-go#8624

@grcevski
grcevski requested a review from a team as a code owner December 1, 2025 18:27
@codecov

codecov Bot commented Dec 1, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 41.17647% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 55.84%. Comparing base (2ae0047) to head (f26ed7e).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
pkg/internal/goexec/structmembers.go 53.84% 4 Missing and 2 partials ⚠️
pkg/internal/ebpf/gotracer/gotracer.go 0.00% 4 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #955   +/-   ##
=======================================
  Coverage   55.84%   55.84%           
=======================================
  Files         255      255           
  Lines       22152    22167   +15     
=======================================
+ Hits        12371    12380    +9     
- Misses       8942     8947    +5     
- Partials      839      840    +1     
Flag Coverage Δ
integration-test 22.84% <0.00%> (-0.05%) ⬇️
integration-test-arm 0.00% <0.00%> (ø)
integration-test-vm-${ARCH}-${KERNEL_VERSION} 0.00% <0.00%> (ø)
k8s-integration-test 2.67% <0.00%> (-0.01%) ⬇️
oats-test 0.00% <0.00%> (ø)
unittests 47.21% <41.17%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@grcevski
grcevski merged commit 7e34dee into open-telemetry:main Dec 2, 2025
66 of 67 checks passed
@grcevski
grcevski deleted the grcp_1.77 branch December 2, 2025 16:07
rafaelroquetto pushed a commit to grafana/opentelemetry-ebpf-instrumentation that referenced this pull request Dec 2, 2025
@MrAlias MrAlias added this to the v0.3.0 milestone Dec 3, 2025
@MrAlias MrAlias mentioned this pull request Dec 3, 2025
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