Skip to content

Conversation

@dmathieu
Copy link
Member

@dmathieu dmathieu commented May 21, 2025

Overview

Added

Changed

  • Jaeger remote sampler's probabilistic strategy now uses the same sampling algorithm as trace.TraceIDRatioBased in go.opentelemetry.io/contrib/samplers/jaegerremote. (Align jaeger probabilistic sampler with trace.TraceIDRatioBased #6892)

  • Switched the default for OTEL_SEMCONV_STABILITY_OPT_IN to emit the v1.26.0 semantic conventions by default in the following modules.

    • go.opentelemetry.io/contrib/instrumentation/github.com/emicklei/go-restful/otelrestful
    • go.opentelemetry.io/contrib/instrumentation/github.com/gin-gonic/gin/otelgin
    • go.opentelemetry.io/contrib/instrumentation/github.com/gorilla/mux/otelmux
    • go.opentelemetry.io/contrib/instrumentation/github.com/labstack/echo/otelecho
    • go.opentelemetry.io/contrib/instrumentation/net/http/httptrace/otelhttptrace
    • go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp
      The OTEL_SEMCONV_STABILITY_OPT_IN=http/dup environment variable can be still used to emit both the v1.20.0 and v1.26.0 semantic conventions.
      It is however impossible to emit only the 1.20.0 semantic conventions, as the next release will drop support for that environment variable. (Switch the semconv migration default to new #6899)
  • Improve performance by reducing allocations for http request when using OTEL_SEMCONV_STABILITY_OPT_IN=http/dup in the modules below. (improve performance for attributes http/dup #7180)

    • go.opentelemetry.io/contrib/instrumentation/github.com/emicklei/go-restful/otelrestful
    • go.opentelemetry.io/contrib/instrumentation/github.com/gin-gonic/gin/otelgin
    • go.opentelemetry.io/contrib/instrumentation/github.com/gorilla/mux/otelmux
    • go.opentelemetry.io/contrib/instrumentation/github.com/labstack/echo/otelecho
    • go.opentelemetry.io/contrib/instrumentation/net/http/httptrace/otelhttptrace
    • go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp
  • Update the Jaeger remote sampler to use "github.com/jaegertracing/jaeger-idl/proto-gen/api_v2" in go.opentelemetry.io/contrib/samplers/jaegerremote. (chore: use jaeger-idl/proto-gen/api_v2 #7061)

  • Improve performance by reducing allocations in the gRPC stats handler in go.opentelemetry.io/contrib/instrumentation/google.golang.org/grpc/otelgrpc. (otelgrpc: reduce allocation for stats handler #7186)

  • Update http.route attribute to support request.Pattern in go.opentelemetry.io/contrib/instrumentation/github.com/gorilla/mux/otelmux. (feat(otelmux): update http.route attribute to support request.Pattern #7108)

  • Change the default span name to be GET /path so it complies with the HTTP semantic conventions in go.opentelemetry.io/contrib/instrumentation/github.com/gin-gonic/gin/otelgin. (otelgin: update default span name to align with semantic conventions #6381)

  • Set url.scheme attribute to the request URL.Scheme when possible for HTTP client metrics in go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp. (otelhttp: Correctly set url.scheme attr in http client metrics. #6938)

  • The semantic conventions have been upgraded from v1.17.0 to v1.30.0 in go.opentelemetry.io/contrib/instrumentation/google.golang.org/grpc/otelgrpc. (Upgrade semantic conventions in otelgrpc from v1.17.0 to v1.30.0 #7270)

    • All net.peer.* and net.host.* attributes are now set to correct server.* attributes.
    • No net.socket.* attributes are set.
  • Only sample spans when Sampled=1 in go.opentelemetry.io/contrib/propagators/aws/xray. (propagators/aws/xray: sample only when Sampled=1 #7318)

Fixed

Removed

@dmathieu dmathieu added the Unlock Released Changelog PRs that need to change released section in CHANGELOG.md label May 21, 2025
@codecov
Copy link

codecov bot commented May 21, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.6%. Comparing base (ea9b739) to head (3771bad).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##            main   #7350     +/-   ##
=======================================
- Coverage   81.6%   81.6%   -0.1%     
=======================================
  Files        205     205             
  Lines      18231   18231             
=======================================
- Hits       14888   14885      -3     
- Misses      2922    2924      +2     
- Partials     421     422      +1     
Files with missing lines Coverage Δ
detectors/aws/ec2/version.go 60.0% <100.0%> (ø)
detectors/aws/ecs/version.go 60.0% <100.0%> (ø)
detectors/aws/eks/version.go 60.0% <100.0%> (ø)
detectors/gcp/version.go 60.0% <100.0%> (ø)
...github.com/aws/aws-lambda-go/otellambda/version.go 100.0% <100.0%> (ø)
...thub.com/aws/aws-sdk-go-v2/otelaws/test/version.go 100.0% <100.0%> (ø)
...on/github.com/aws/aws-sdk-go-v2/otelaws/version.go 100.0% <100.0%> (ø)
...om/emicklei/go-restful/otelrestful/test/version.go 100.0% <100.0%> (ø)
...hub.com/emicklei/go-restful/otelrestful/version.go 100.0% <100.0%> (ø)
...n/github.com/gin-gonic/gin/otelgin/test/version.go 100.0% <100.0%> (ø)
... and 23 more

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dmathieu dmathieu marked this pull request as ready for review May 21, 2025 14:22
@dmathieu dmathieu requested a review from a team as a code owner May 21, 2025 14:22
dmathieu and others added 2 commits May 21, 2025 16:29
Co-authored-by: Robert Pająk <[email protected]>
Co-authored-by: Robert Pająk <[email protected]>
Copy link
Contributor

@MrAlias MrAlias left a comment

Choose a reason for hiding this comment

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

This does not bump any contrib repositories. The versions.yaml file needs to be updated and the release make target run.

@dmathieu dmathieu changed the title Release v1.36.0/v0.61.0/v0.30.0/v0.16.0/v0.11.0/v0.9.0/v0.8.0 Release v1.36.0/v0.61.0/v0.30.0/v0.11.0/v0.9.0/v0.8.0 May 21, 2025
@dmathieu dmathieu changed the title Release v1.36.0/v0.61.0/v0.30.0/v0.11.0/v0.9.0/v0.8.0 Release v1.36.0/v0.61.0/v0.30.0/v0.16.0/v0.11.0/v0.9.0/v0.8.0 May 21, 2025
@dmathieu
Copy link
Member Author

Yes, sorry, my bad. That has been done now.

@dmathieu dmathieu requested a review from MrAlias May 21, 2025 14:43
@MrAlias
Copy link
Contributor

MrAlias commented May 21, 2025

Let's wait for the resolution of open-telemetry/opentelemetry-go#6803 so we can depend on the latest releases with retractions.

@dmathieu
Copy link
Member Author

This is now using log/0.12.2.

@pellared pellared merged commit 934fb13 into open-telemetry:main May 22, 2025
37 of 44 checks passed
@MrAlias MrAlias added this to the v1.36.0 milestone May 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Unlock Released Changelog PRs that need to change released section in CHANGELOG.md

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants